- User Since
- Apr 14 2015, 9:17 PM (138 w, 6 d)
Sat, Dec 9
I've pushed this patch with a few changes (don't want to wait the last moment before the string freeze).
Thu, Dec 7
Hi @abetts , we need a nice dialogue that shows the password entry for unlocking a vault. You have the current proposal screenshot above.
Tue, Dec 5
Since VDG has not responded. Can you remove the padding and make the name bold?
Fri, Dec 1
Why don't you set up the preferred size for the dialogue, and allow the label to be multi-line?
Thu, Nov 30
@ngraham I can not reproduce the text overflow bud you are getting. I'm not setting any size-related things, so the bug is probably in the upstream (libplasma or above)
- Reverted the 'create a new vault button' to be a proper button
Wed, Nov 29
Almost mergeable - I just have a few nit-picks:
I was on a vacation, sorry. I'll test everything today and merge it if I don't find a problem.
Fri, Nov 17
The activities cache is a private class meant to communicate directly to KAMD. The signals it contains are in the form (activity id, something) and are meant to be private. The KActivities::Info class was meant to wrap these signals into a nice OO API where an activity object (Info) sends signals that a particular property has changed.
Nov 3 2017
While I like the idea behind the patch, I'd rather have what David proposed. Namely, d_ptr is a smart pointer, but I don't think it should be this smart - creating specializations for different types' destructors should not really be a part of it.
Oct 29 2017
Also, not sure adding a signal that has one user only to a shared library is the best approach, especially since .h says this:
Oct 25 2017
This one is almost mergeable.
I was not clear enough probably - I meant that "Have a default backend" does not warrant a change to the UI. Especially when it makes the UI more complex.
Oct 24 2017
Do I need to relogin to make a kded module show up?
Having a default backend (IMO) has nothing to do with the UI. What is the rationale behind changing a combo-box (a widget designed for multiple choice questions) to a button?
Oct 23 2017
Oct 11 2017
Somehow I didn't submit the comment when I wrote it. See above.
Sep 28 2017
Sep 23 2017
Sep 8 2017
Increasing the result limit
Removed model test
Sep 7 2017
p.s. This is how I restart everything:
Another one to test:
- fixed a small bug related to fav adding
- added debugging output for easier testing (when you encounter a bug, send me an email with plasma output)
Sep 5 2017
Fixing the ordering
Sep 4 2017
Post-migration sorting still feels completely random - not sure how solid that feeling is though.
Sep 2 2017
The last patch supports Kickoff, Kicker and Dash. I've tested what I can, found an small issue in KAMD (update KAMD to latest master before testing) and it looks like we are on the finish line.
Sep 1 2017
Good news: There's actually nothing to do on the favorite ids, Kicker already uses
KService::storageId as favorite id, which will be the menuId when it's available.
Aug 31 2017
This might be a hint:
Regarding Kickoff - it looks like there are some problems in its model handling:
The initial sorting of the migrated superset seems random. How is it done?
Aug 30 2017
With icons (designed after the Dolphin plugin for activity linking - to at least be consistent with something :) )
No Kickoff support: Personally I think we should actually migrate all applets at once.
Aug 17 2017
Aug 16 2017
p.s. Transitioned items go to all activities and the documents are supported in the current patch.
Hmm I think the Kickoff thing might have been a misunderstanding, or maybe I was full of crap then?
The result of the migration is now the superset of all faves, but sorting is a bit weird (maybe this is the best possible?)
Jul 15 2017
This was a referse-murphy - the second I posted that I can not find the reason behind the crash, I managed. Can you try the current revision whether it crashes or not?
I've finally managed to reproduce the crash (clean install, clean build, clean everything). I still do not see what is the cause of it...
Jul 11 2017
Ignore my comment above, I'm colour-blind today :D
It is undefined behaviour to extend std:: namespace in this way. You should go for this instead:
Jul 3 2017
True, I haven't updated the branch - had more than a few sub-branches since the last update.
Jul 2 2017
I've just added another test - having a custom kicker plasmoidsetupscript and putting kicker to be the default launcher on a blank user loads the favourites from said plasmoidsetupscript.
Jun 26 2017
I realized I can be quite thick.
Jun 23 2017
May 24 2017
It would be nicer if we had a better way to do this instead of having a chunk of code copy-pasted all over the place.
May 22 2017
This is meant to fix the https://bugs.kde.org/show_bug.cgi?id=379414
May 17 2017
May 6 2017
Not sure I'd rush it for 5.11. I'll probably have to tweak the formula coefficients for the score - how much deterioration over time it should have, ... (this is the main reason I'm making the graphs - to be able to easier debug the stats - having it in a user application is less important.
This is a good point. *