Reworked, by reshuffling the dialog text, instead of renaming the buttons
- Queries
- All Stories
- Search
- Advanced Search
Advanced Search
Dec 13 2018
Dec 12 2018
Will have to rework, I missed some cases(the dialog might as to activate. deactivate or "activate some, deactivate others"), it is not as simple as I thought.
Accessibility bugfixes cherry picked from master
Dec 11 2018
Messed up a little and this got landed together with another patch https://cgit.kde.org/plasma-desktop.git/commit/?id=40b9e34ced6b421c8585f304d91a2da37f40c4d6
Dec 8 2018
Added comment for workaround
Dec 7 2018
Dec 3 2018
Oct 9 2018
Oct 6 2018
Go ahead, I'm fine with the license change.
Mar 8 2018
Moved to 5.12
Feb 28 2018
I can't put to 5.12 as it seems https://phabricator.kde.org/D9325 didn't get into the 5.12 branch, and I don't want two versions of this patch. May I cherry-pick that commit to 5.12 and apply this patch?
Yes, I will push first to 5.12 both of them
It adds a new entry, doesn't change an existing one. If that is not there, I see no problems, the ScreenMapper::readDisabledScreensMap() will not initialize the m_itemsOnDisabledScreensMap. That map was not saved previously, thus the folderview behaving differently after a restart, as the map was lost.
Ping?
Ping?
Feb 21 2018
Feb 19 2018
Feb 12 2018
Feb 10 2018
Yes, I have commit access since some time. ;)
Feb 9 2018
Jan 26 2018
Jan 25 2018
Make the code more clear, by renaming a local variable.
Jan 23 2018
Change code according to reviewer's requests
Jan 6 2018
Sorry about it.
Dec 22 2017
Dec 18 2017
Dec 15 2017
Fixed issues pointed out by Laurent.
I will merge this latest on Monday if there is no further objection
Dec 14 2017
In D9300#179031, @broulik wrote:Screen mapper is a singleton type, so it won't cause problems when one screen adds a mapping, it will propagate to the other containments automagically? Before we had plasmoid.configuration which signals changes but C++ does not, obviously.
To be honest, I'm not sure the benefit of a nice API warrants the rest of the ugliness, as some parts still rely on using a string for URL (FolderModel::url) and makes the unit tests also uglier.
Connect to screenammper only if used as a containment
Dec 13 2017
I think it is better to store once, than 2-3 times (per desktop) as the data is the same. I'm not sure if the corona level is the best, although I don't see big problems with it.
Add const
Dec 12 2017
Nov 28 2017
Rebased
Nov 27 2017
@davidedmundson Any comments? If not, I will push this tomorrow.
Nov 24 2017
Guard screenmapper usage
Nov 23 2017
Guard m_screenMapper usage
Milian, please also update the commit description (the last paragraph needs to go away).