[plasmashell] [Bug 483299] Make it easier to exit panel editing mode

2024-05-09 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=483299

Nate Graham  changed:

   What|Removed |Added

   Version Fixed In||6.1
  Latest Commit||https://invent.kde.org/plas
   ||ma/plasma-desktop/-/commit/
   ||52d352fb32abd876fdfd51fd5c9
   ||ed28ce73b0442
 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED

--- Comment #3 from Nate Graham  ---
Git commit 52d352fb32abd876fdfd51fd5c9ed28ce73b0442 by Nate Graham.
Committed on 10/05/2024 at 00:42.
Pushed by ngraham into branch 'master'.

PanelConfiguration: make it more obvious how you close it

Right now, there's no clear way to close the panel configuration dialog.
It lacks both an OK or Done button like you'd see in a dialog, and also
a close button in the fake titlebar like you'd see in a real window.
The dialog also doesn't disappear on focus loss like the old UI did.

This isn't ideal. Let's address it by adding a "Done" button to the
bottom-trailing position, where people generally expect to see buttons
to close dialogs.

I chose this over a close button in the fake titlebar because a close
button (especially a red one, as the Breeze icon is) might be seen more
negatively, i.e. "throw away settings". Whereas a "Done" button with
a checkmark is clearly an affirmative action that will accept and save
any changed settings.

To make room, the Delete Panel button is moved to the other side of the
footer, and the shortcut UI is added into the body of the dialog, where
the thickness control lives.

---

Thanks to @eatsu for starting this off in
https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2021.

This is an intentionally minimal change to fix a targeted UX issue; let's try
to avoid redesigning the entire dialog or else we'll go in circles like we did
there.

Co-authored-by: Taro Tanaka 
Related: bug 486703
FIXED-IN: 6.1

@teams/vdg @teams/usability 

| Before | After |
| -- | - |
| ![Before](/uploads/e74e84963544a4d4a5e35ed4245ed951/Before.jpeg) |
![After](/uploads/17e217a34b5ff5809491764afcb5ef32/After.jpeg) |

M  +43   -26   desktoppackage/contents/configuration/PanelConfiguration.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/52d352fb32abd876fdfd51fd5c9ed28ce73b0442

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 483299] Make it easier to exit panel editing mode

2024-05-07 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=483299

--- Comment #2 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2240

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 483299] Make it easier to exit panel editing mode

2024-05-07 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=483299

Nate Graham  changed:

   What|Removed |Added

   See Also||https://bugs.kde.org/show_b
   ||ug.cgi?id=486703

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 483299] Make it easier to exit panel editing mode

2024-03-13 Thread postix
https://bugs.kde.org/show_bug.cgi?id=483299

postix  changed:

   What|Removed |Added

 CC||pos...@posteo.eu

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 483299] Make it easier to exit panel editing mode

2024-03-13 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=483299

Nate Graham  changed:

   What|Removed |Added

 Status|REPORTED|ASSIGNED
   Keywords||usability
   Severity|normal  |wishlist
 CC||n...@kde.org
 Ever confirmed|0   |1

--- Comment #1 from Nate Graham  ---
In progress with
https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2021.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 483299] Make it easier to exit panel editing mode

2024-03-11 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=483299

Bug Janitor Service  changed:

   What|Removed |Added

   Keywords||qt6

-- 
You are receiving this mail because:
You are watching all bug changes.