ElisaProject
ActivePublic

Recent Activity

Sun, Dec 9

mgallien closed D17163: Display embedded cover images.
Sun, Dec 9, 7:49 PM · Elisa

Tue, Dec 4

astippich updated the task description for T8196: KFileMetadata: String lists. How do clients cope?.
Tue, Dec 4, 7:42 PM · Dolphin, Elisa, Frameworks, Baloo

Sun, Dec 2

astippich updated the task description for T8196: KFileMetadata: String lists. How do clients cope?.
Sun, Dec 2, 7:05 PM · Dolphin, Elisa, Frameworks, Baloo

Wed, Nov 28

mgallien added a project to D17163: Display embedded cover images: Elisa.
Wed, Nov 28, 8:56 PM · Elisa

Tue, Nov 27

astippich placed T7576: port to controls2 up for grabs.
Tue, Nov 27, 7:38 PM · Elisa
astippich placed T7884: file browser up for grabs.
Tue, Nov 27, 7:37 PM · Elisa
astippich reassigned T6255: Add support for cover images embedded in tracks from astippich to mgallien.
Tue, Nov 27, 7:37 PM · Elisa

Sun, Nov 25

mgallien closed D16302: improve database performance and reliability.
Sun, Nov 25, 7:27 PM · Elisa
mgallien updated the summary of D16302: improve database performance and reliability.
Sun, Nov 25, 7:25 PM · Elisa

Sat, Nov 24

astippich added a comment to D16944: Port application menu to qqc2.

Sorry, forgot to write a comment. This is not meant to be merged right now, I wanted to start a discussing which type of menu Elisa should use, e.g. the item-based (standard qqc2) or the native one (Qt.labs).
Both currently have issues which need to be fixed first.
I have at least an idea what could be done to fix the item-based solution, and I have no clue what's going wrong with the native solution.

Sat, Nov 24, 7:46 PM · Elisa
astippich accepted D16302: improve database performance and reliability.

Working good so far. Memory usage seems to have dropped a little with my smaller music collection. Tracks with an empty artist tag are also correctly displayed in the all tracks view.

Sat, Nov 24, 6:47 PM · Elisa

Tue, Nov 20

mgallien added a comment to D16302: improve database performance and reliability.

Since this revision is quite large, it's probably difficult to review it properly. I'm going to let it run a little and report back in case I find any issues.

Tue, Nov 20, 9:43 PM · Elisa
mgallien updated the diff for D16302: improve database performance and reliability.
  • fix errors when inserting multiple albums with same title
  • really cache only artists and not the other people
  • fix import of tracks with paths containing particular character like { }
Tue, Nov 20, 9:35 PM · Elisa

Mon, Nov 19

ngraham added a comment to D16944: Port application menu to qqc2.

After the tests I had done, I am convinced that we should first fix the stack used by Elisa before pushing this kind of changes.

Mon, Nov 19, 8:30 PM · Elisa
mgallien requested changes to D16944: Port application menu to qqc2.

The menu is no longer a sub window manage by the windowing system. Is this really a good idea ?

Mon, Nov 19, 8:26 PM · Elisa

Sun, Nov 18

ssteffen removed a watcher for Elisa: ssteffen.
Sun, Nov 18, 12:23 PM
astippich added a project to D16953: Port notification to qqc2: Elisa.
Sun, Nov 18, 11:39 AM · Elisa
astippich added a project to D16952: Port browsing delegates to qqc2: Elisa.
Sun, Nov 18, 11:39 AM · Elisa
astippich added a project to D16950: Port navigation bar to qqc2: Elisa.
Sun, Nov 18, 11:39 AM · Elisa
astippich added a project to D16948: Port qml media playlist entirely to qqc2: Elisa.
Sun, Nov 18, 11:38 AM · Elisa
astippich added a project to D16944: Port application menu to qqc2: Elisa.
Sun, Nov 18, 11:38 AM · Elisa
astippich requested review of D16975: Move all platform-agnostic files related to scanning into one folder.
Sun, Nov 18, 11:37 AM · Elisa

Sat, Nov 17

mgallien added a comment to D16302: improve database performance and reliability.

Since this revision is quite large, it's probably difficult to review it properly. I'm going to let it run a little and report back in case I find any issues.

Sat, Nov 17, 8:48 PM · Elisa
astippich updated the diff for D16194: move metadata scanning to trackslistener.
  • rebase
  • cleanup
Sat, Nov 17, 4:13 PM · Elisa
astippich added a comment to D16302: improve database performance and reliability.

Since this revision is quite large, it's probably difficult to review it properly. I'm going to let it run a little and report back in case I find any issues.

Sat, Nov 17, 3:17 PM · Elisa
mgallien updated the diff for D16302: improve database performance and reliability.
  • allow tracks without artist
Sat, Nov 17, 3:14 PM · Elisa
mgallien updated the diff for D16302: improve database performance and reliability.
  • reduce overhead from AllTracksModel
  • reduce overhead of AllAlbumsModel and make it synchronous
  • do not list files too early from Baloo
  • keep all files discovered from a source and fix a minor related issue
  • add a test for DatabaseInterface::askRestoredTracks
  • add albums and artists cache and improve cache usage
  • improve management of files not modified since last scan of music files
Sat, Nov 17, 2:14 PM · Elisa
astippich added a comment to D15195: consolidate handling of title fallback into MusicAudioTrack.

ping

Sat, Nov 17, 9:12 AM · Elisa
astippich closed D16193: add a test for tracks with empty artist tag in database.
Sat, Nov 17, 9:11 AM · Elisa
astippich closed D13872: create a FlatButtonWithToolTip component for MediaPlayerControl.
Sat, Nov 17, 9:11 AM · Elisa
astippich closed D16362: use rating and comment stored in metadata tags.
Sat, Nov 17, 9:10 AM · Elisa
astippich added a comment to D16362: use rating and comment stored in metadata tags.

thanks

Sat, Nov 17, 9:10 AM · Elisa
mgallien accepted D13872: create a FlatButtonWithToolTip component for MediaPlayerControl.
Sat, Nov 17, 8:38 AM · Elisa
mgallien accepted D16362: use rating and comment stored in metadata tags.

Sorry for the delay

Sat, Nov 17, 8:36 AM · Elisa
astippich claimed T6255: Add support for cover images embedded in tracks.
Sat, Nov 17, 8:31 AM · Elisa

Tue, Nov 13

mgallien updated the summary of D16302: improve database performance and reliability.
Tue, Nov 13, 6:39 AM · Elisa
mgallien retitled D16302: improve database performance and reliability from [WIP] improve database performance and reliability to improve database performance and reliability.
Tue, Nov 13, 6:38 AM · Elisa
mgallien updated the diff for D16302: improve database performance and reliability.
  • reduce overhead from AllTracksModel
  • reduce overhead of AllAlbumsModel and make it synchronous
  • do not list files too early from Baloo
  • keep all files discovered from a source and fix a minor related issue
  • add a test for DatabaseInterface::askRestoredTracks
  • add albums and artists cache and improve cache usage
  • improve management of files not modified since last scan of music files
Tue, Nov 13, 6:38 AM · Elisa

Nov 4 2018

astippich retitled D13872: create a FlatButtonWithToolTip component for MediaPlayerControl from create a ButtonWithToolTip component for MediaPlayerControl to create a FlatButtonWithToolTip component for MediaPlayerControl.
Nov 4 2018, 10:50 AM · Elisa
astippich added a comment to D13872: create a FlatButtonWithToolTip component for MediaPlayerControl.

D16659 and its dependent revision will allow to remove custom styling in the future

Nov 4 2018, 10:48 AM · Elisa
astippich updated the diff for D13872: create a FlatButtonWithToolTip component for MediaPlayerControl.
  • rename to FlatButtonWithToolTip
Nov 4 2018, 10:47 AM · Elisa

Nov 2 2018

astippich added inline comments to D16362: use rating and comment stored in metadata tags.
Nov 2 2018, 8:06 PM · Elisa
astippich added a comment to D13872: create a FlatButtonWithToolTip component for MediaPlayerControl.

I lack time until next week for a proper review.

Nov 2 2018, 8:01 PM · Elisa
mgallien added a comment to D13872: create a FlatButtonWithToolTip component for MediaPlayerControl.

see D16284

I have been working on this subject to help come this diff come to a conclusion. Thanks for your work on this as it is important.

I believe that we have been overusing ToolButton at places just to get a flat look instead of doing the proper job (i.e. customizing the look of a Button component).

I also better understand (thanks to D16284) why we get icon+text. I believe that this will not change as it is done on purpose to maximize compatibility with QWidget equivalent of ToolButton. We are the guilty ones here given we had used ToolButton components in places where we expected a custom behavior (i.e. no text).

I have a private (at the moment) branch with no QQCv1 components. There are several regressions. I had kept overusing ToolButton like in master branch.

I would like to try to come up with a long term and maintainable solution for each of our button cases:

  • flat button with text and icon ;
  • flat button with tooltip and icon ;
  • flat round button with icon ;

    The result should be fully usable with keyboard only, mouse+keyboard and touch screens. That means that ToolButton as is is not the solution because the focus behavior is not going to work for keyboard only usage. Our solution should work with org.kde.desktop and Fusion style (at least).

I've gone ahead and factored out the current button code that was copied all over in MediaPlayerControl. As it is now, it's mostly a refactoring only with the exception that actions are now used.
Imho we don't need a round button anymore.
If you agree on the implementation, I will roll this out to all qqc1 toolbuttons in the views and playlist.

Nov 2 2018, 9:37 AM · Elisa
mgallien added inline comments to D16362: use rating and comment stored in metadata tags.
Nov 2 2018, 9:14 AM · Elisa

Nov 1 2018

astippich added a comment to D13872: create a FlatButtonWithToolTip component for MediaPlayerControl.

see D16284

I have been working on this subject to help come this diff come to a conclusion. Thanks for your work on this as it is important.

I believe that we have been overusing ToolButton at places just to get a flat look instead of doing the proper job (i.e. customizing the look of a Button component).

I also better understand (thanks to D16284) why we get icon+text. I believe that this will not change as it is done on purpose to maximize compatibility with QWidget equivalent of ToolButton. We are the guilty ones here given we had used ToolButton components in places where we expected a custom behavior (i.e. no text).

I have a private (at the moment) branch with no QQCv1 components. There are several regressions. I had kept overusing ToolButton like in master branch.

I would like to try to come up with a long term and maintainable solution for each of our button cases:

  • flat button with text and icon ;
  • flat button with tooltip and icon ;
  • flat round button with icon ;

    The result should be fully usable with keyboard only, mouse+keyboard and touch screens. That means that ToolButton as is is not the solution because the focus behavior is not going to work for keyboard only usage. Our solution should work with org.kde.desktop and Fusion style (at least).
Nov 1 2018, 7:43 PM · Elisa
astippich updated the summary of D13872: create a FlatButtonWithToolTip component for MediaPlayerControl.
Nov 1 2018, 7:39 PM · Elisa
astippich updated the diff for D13872: create a FlatButtonWithToolTip component for MediaPlayerControl.
  • do not use toolbutton
Nov 1 2018, 7:37 PM · Elisa
astippich added a comment to D14018: [WIP] Make labels in header bar clickeable.

I think you need to install the Qt5CorePrivate development files

Nov 1 2018, 3:38 PM · Elisa

Oct 31 2018

astippich added a comment to D13872: create a FlatButtonWithToolTip component for MediaPlayerControl.

see D16284

I have been working on this subject to help come this diff come to a conclusion. Thanks for your work on this as it is important.

I believe that we have been overusing ToolButton at places just to get a flat look instead of doing the proper job (i.e. customizing the look of a Button component).

I also better understand (thanks to D16284) why we get icon+text. I believe that this will not change as it is done on purpose to maximize compatibility with QWidget equivalent of ToolButton. We are the guilty ones here given we had used ToolButton components in places where we expected a custom behavior (i.e. no text).

Oct 31 2018, 9:38 PM · Elisa