1. 21 Dec, 2018 1 commit
    • Michael Pyne's avatar
      Fix the "Show Columns" menu to show correct state on startup. · f9721878
      Michael Pyne authored
      The "Show Columns" menu would previously have a checkbox entered for
      every possible column for the playlist shown by default at JuK startup,
      even though most columns were not visible.
      
      This turned out to be because the class that reads the column
      configuration (Playlist::SharedSettings) was not being used to generate
      the Show Columns menu options. Instead the Playlist's own (defaulted)
      values were used.
      
      At this point in execution we can't force SharedSettings->apply() to run
      because the Playlist is not fully setup. Instead we expose the config
      values and use those to generate the menu.
      f9721878
  2. 10 Feb, 2018 1 commit
    • Michael Pyne's avatar
      Extract Playlist::SharedSettings into a separate file. · 90a6b862
      Michael Pyne authored
      This helps declutter playlist.cpp and even makes it seem a bit easier to
      read through, I think.
      
      I also tried to fix a couple of outstanding bugs along the way, for
      instance reordering columns should work now, although there's still a
      lot of improvements that need to be made here.
      90a6b862
  3. 01 Feb, 2018 1 commit
    • Michael Pyne's avatar
      Extract Playlist::SharedSettings into a separate file. · f3171c17
      Michael Pyne authored
      This helps declutter playlist.cpp and even makes it seem a bit easier to
      read through, I think.
      
      I also tried to fix a couple of outstanding bugs along the way, for
      instance reordering columns should work now, although there's still a
      lot of improvements that need to be made here.
      f3171c17