[Breeze] [Bug 447288] Assigning a menu to an action gives it excessive left padding

2022-01-18 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=447288

Nate Graham  changed:

   What|Removed |Added

   Version Fixed In|5.25|5.24

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

[Breeze] [Bug 447288] Assigning a menu to an action gives it excessive left padding

2022-01-18 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=447288

Nate Graham  changed:

   What|Removed |Added

  Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
   |ma/breeze/commit/47a015cde2 |ma/breeze/commit/7519dec9b5
   |951af65fe17c20bdbe0034cb483 |5fd58662a7f79c157ec032acca6
   |59d |a2f

--- Comment #6 from Nate Graham  ---
Git commit 7519dec9b55fd58662a7f79c157ec032acca6a2f by Nate Graham, on behalf
of Fushan Wen.
Committed on 18/01/2022 at 20:30.
Pushed by ngraham into branch 'Plasma/5.24'.

breezestyle: Remove extra margin for InstantPopup

Assigning a menu with instant popup mode to an action gives it
excessive left padding. Removing the extra margin fixes it.


(cherry picked from commit 47a015cde2951af65fe17c20bdbe0034cb48359d)

M  +0-6kstyle/breezestyle.cpp

https://invent.kde.org/plasma/breeze/commit/7519dec9b55fd58662a7f79c157ec032acca6a2f

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

[Breeze] [Bug 447288] Assigning a menu to an action gives it excessive left padding

2022-01-15 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=447288

Fushan Wen  changed:

   What|Removed |Added

   Version Fixed In||5.25

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

[Breeze] [Bug 447288] Assigning a menu to an action gives it excessive left padding

2022-01-14 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=447288

Fushan Wen  changed:

   What|Removed |Added

  Latest Commit||https://invent.kde.org/plas
   ||ma/breeze/commit/47a015cde2
   ||951af65fe17c20bdbe0034cb483
   ||59d
 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED

--- Comment #5 from Fushan Wen  ---
Git commit 47a015cde2951af65fe17c20bdbe0034cb48359d by Fushan Wen.
Committed on 15/01/2022 at 05:12.
Pushed by fusionfuture into branch 'master'.

breezestyle: Remove extra margin for InstantPopup

Assigning a menu with instant popup mode to an action gives it
excessive left padding. Removing the extra margin fixes it.

M  +0-6kstyle/breezestyle.cpp

https://invent.kde.org/plasma/breeze/commit/47a015cde2951af65fe17c20bdbe0034cb48359d

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

[Breeze] [Bug 447288] Assigning a menu to an action gives it excessive left padding

2022-01-14 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=447288

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

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

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

[Breeze] [Bug 447288] Assigning a menu to an action gives it excessive left padding

2021-12-20 Thread Fushan Wen
https://bugs.kde.org/show_bug.cgi?id=447288

Fushan Wen  changed:

   What|Removed |Added

 CC||qydwhotm...@gmail.com

--- Comment #3 from Fushan Wen  ---
I tried to fix the big by reading Breeze code, but found nothing and I should
say the coding style is weird (maybe ancient). Setting subcontrol-position:
right center; will fix the alignment issue, but the arrow position becomes
incorrect.

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

[Breeze] [Bug 447288] Assigning a menu to an action gives it excessive left padding

2021-12-20 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=447288

Nate Graham  changed:

   What|Removed |Added

Version|21.04.2 |5.23.4
   Assignee|gwenview-bugs-n...@kde.org  |plasma-b...@kde.org
 CC||noaha...@gmail.com
   Keywords||regression
  Component|general |QStyle
Summary|Open with menu entry in the |Assigning a menu to an
   |sidebar for file operation  |action gives it excessive
   |section has empty space |left padding
   |before it.  |
Product|gwenview|Breeze

--- Comment #2 from Nate Graham  ---
This seems to be a bug in the Breeze style. Somehow giving this action a menu
causes it to be indented too much on the left. Removing the
`mOpenWithAction->setMenu(menu);` line gets rid of the extra margin.

See
https://invent.kde.org/graphics/gwenview/-/blob/master/app/fileopscontextmanageritem.cpp#L192

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