D4918: [Folder View] Increase delegate action size to smallMedium in icon mode

2017-03-07 Thread Kai Uwe Broulik
broulik abandoned this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4918 To: broulik, #plasma, hein Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D4918: [Folder View] Increase delegate action size to smallMedium in icon mode

2017-03-07 Thread Eike Hein
hein added a comment. After trying it I still don't like smallMedium for icon mode :(. Just seems too huge (also rather dislike the icons, and the press color clashes with the selected frame state ...). REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4918

D4918: [Folder View] Increase delegate action size to smallMedium in icon mode

2017-03-06 Thread Eike Hein
hein added a comment. I'll check out the new icons tomorrow and then report back (nearing 7 AM here ...). REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4918 To: broulik, #plasma, hein Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg,

D4918: [Folder View] Increase delegate action size to smallMedium in icon mode

2017-03-06 Thread Kai Uwe Broulik
broulik added a comment. Do you already have the new icons I pushed this morning? They're gray if not hovered so should be less intrusive. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4918 To: broulik, #plasma, hein Cc: plasma-devel, progwolff,

D4918: [Folder View] Increase delegate action size to smallMedium in icon mode

2017-03-06 Thread Eike Hein
hein added a comment. I think it'd feel oppressively big - I'd prefer to keep the visual noise of those knobs small I guess? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4918 To: broulik, #plasma, hein Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed,

D4918: [Folder View] Increase delegate action size to smallMedium in icon mode

2017-03-06 Thread Kai Uwe Broulik
broulik added a comment. Oh, sorry, I misread your comment. Imho in icon mode it should definitely be smallMedium. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4918 To: broulik, #plasma, hein Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed,

D4918: [Folder View] Increase delegate action size to smallMedium in icon mode

2017-03-06 Thread Eike Hein
hein added a comment. Wait, isn't this the reverse? You wanted smallMedium for listview mode for covering and small for icon mode (as previous, after my comment), no? :) REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4918 To: broulik, #plasma, hein Cc:

D4918: [Folder View] Increase delegate action size to smallMedium in icon mode

2017-03-06 Thread Kai Uwe Broulik
broulik updated this revision to Diff 12237. broulik retitled this revision from "[Folder View] Increase delegate action size to smallMedium" to "[Folder View] Increase delegate action size to smallMedium in icon mode". broulik edited the summary of this revision. broulik edited the test plan

D4918: [Folder View] Increase delegate action size to smallMedium

2017-03-03 Thread Eike Hein
hein added a comment. Maybe keep it small in icon view mode though ...? REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4918 To: broulik, #plasma, hein Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D4918: [Folder View] Increase delegate action size to smallMedium

2017-03-03 Thread Kai Uwe Broulik
broulik edited the test plan for this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D4918 To: broulik, #plasma, hein Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

D4918: [Folder View] Increase delegate action size to smallMedium

2017-03-03 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY The SVGs in Plasma's theme have a 22x22 size (which is smallMedium). This makes the elements more easily clickable and also improves their