Added context to the diff. Thanks @shubham.
- Queries
- All Stories
- Search
- Advanced Search
Advanced Search
Jan 11 2019
Ok i will start using archanist ASAP
In D17994#391102, @ashwind wrote:I used archanist to apply the patch. It is working fine for me.
I used archanist to apply the patch. It is working fine for me.
Made the shortcut configurable. I actually did a "git pull" and updated the feature out from a fresh repo. It should work now.
Ok, I will look into it.
Jan 10 2019
Thanks for your patch.
Thanks! I'll let @mgallien land this for you when he's ready.
Name: Ashwin Dhakaita
Email: ashwingpdhakaita@gmail.com
In the meantime, @ashwind, we will need you to provide your full name and email address so we can land this patch with the correct authorship information.
Thank you, @mgallien. I will apply for one after sometime then.
In D18017#390492, @ashwind wrote:I don't have a developer account. Please land it for me.
Actually, I am new to KDE and don't know about Developer Account. I will sign up for one.
I don't have a developer account. Please land it for me.
Actually, I am new to KDE and don't know about Developer Account. I will sign up for one.
In D18017#390169, @mgallien wrote:Nice work. Thanks for your help.
Jan 9 2019
In D18017#389338, @ngraham wrote:Oops, I should have realized that!
BTW @mgallien, it's off-topic for this patch, but I wanted to express my happiness over how good Elisa is these days. It's becoming really excellent. The full-screen view is so beautiful and compelling! I plan to contribute a lot in 2019. Once we get https://bugs.kde.org/show_bug.cgi?id=401969, I'm ditching Lollypop and moving to it full-time.
Nice work. Thanks for your help.
Well, I made the shortcuts configurable.
Jan 8 2019
Sorry, I will look into it. 😕😕
Oops, I should have realized that!
Thanks for your patch.
In order to better integrate it in the application, it would need to be configurable like other existing shortcuts. In order to do that, you can look at the ElisaApplication class.
Jan 7 2019
Very cool!!! Works great for me.
Jan 6 2019
Thank you Sir 😊
This works great, I love it. It's up to the Elisa devs of course, but this gets a thumbs up from me.
Well I cleared up the trailing whitespaces.
Ok.
Jan 5 2019
Thanks for the patch! There are a lot of unintentional trailing whitespace changes here. Do you think you can clean those up?
Jan 4 2019
Dec 30 2018
Dec 20 2018
Dec 9 2018
Dec 4 2018
Dec 2 2018
Nov 28 2018
Nov 27 2018
Nov 25 2018
Nov 24 2018
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.
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.
Nov 20 2018
In D16302#361032, @astippich wrote: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.
- 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 { }
Nov 19 2018
In D16944#362538, @mgallien wrote: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.
The menu is no longer a sub window manage by the windowing system. Is this really a good idea ?
Nov 18 2018
Nov 17 2018
In D16302#361032, @astippich wrote: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.
- rebase
- cleanup
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.
- allow tracks without artist
- 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
ping
thanks
Sorry for the delay
Nov 13 2018
- 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
Nov 4 2018
D16659 and its dependent revision will allow to remove custom styling in the future
- rename to FlatButtonWithToolTip