Bug 338636 - Panel controller buttons overlap
Summary: Panel controller buttons overlap
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Panel (show other bugs)
Version: master
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: David Edmundson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-08-28 19:25 UTC by Kai Uwe Broulik
Modified: 2014-09-30 22:44 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Buttons overlap (35.95 KB, image/png)
2014-08-28 19:25 UTC, Kai Uwe Broulik 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Kai Uwe Broulik 2014-08-28 19:25:52 UTC 
Created attachment 88478 [details]
Buttons overlap

The Add widget/Add spacer/More options buttons overlap the Location/Height buttons. This is German locale and 180dpi. See screenshot.
Comment 1 Christoph Feck 2014-08-29 00:00:23 UTC 
At 180 dpi, your display has 1538 pixels horizontal resolution?
Comment 2 Kai Uwe Broulik 2014-08-29 05:45:46 UTC 
It's 2560x1440, I cropped the screenshot, sorry.
Comment 3 David Edmundson 2014-09-09 10:27:05 UTC 
It /should /have code to go into icon view at this point.  Clearly it's not working.
I'll have a look and redo it.
Comment 4 Marco Martin 2014-09-26 16:28:23 UTC 
it's definitely working on low resolutions, it probably calculates it wrong (tm) on high dpi
Comment 5 David Edmundson 2014-09-26 16:29:40 UTC 
I did a load of changes on toolbuttons implicitSizes recently. Might be relevant.

Kai, can you retest?
Comment 6 Kai Uwe Broulik 2014-09-27 09:41:04 UTC 
The issue persists with plasma-{framework,workspace,desktop} master
Comment 7 David Edmundson 2014-09-27 09:53:06 UTC 
Reopening.
Comment 8 David Edmundson 2014-09-27 09:54:41 UTC 
I tested this on Vishesh's screen and it looked ok. I'm not sure what makes you so different.
Comment 9 David Edmundson 2014-09-30 22:44:20 UTC 
Kai fixed this.

There were two i18n strings, one was translated one wasn't. Caused an issue.