Bug 365735 - Long category names overlap each other
Summary: Long category names overlap each other
Status: RESOLVED FIXED
Alias: None
Product: Discover
Classification: Applications
Component: discover (show other bugs)
Version: unspecified
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Aleix Pol
URL:
Keywords:
: 367939 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-07-16 10:10 UTC by Bernhard Scheirle
Modified: 2016-08-29 00:23 UTC (History)
1 user (show)

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


Attachments
Screenshot of the bug (458.91 KB, image/png)
2016-07-16 10:11 UTC, Bernhard Scheirle
Details
Screenshot of the fix (189.80 KB, image/png)
2016-07-16 10:11 UTC, Bernhard Scheirle
Details
Patch (535 bytes, patch)
2016-07-16 10:12 UTC, Bernhard Scheirle
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Bernhard Scheirle 2016-07-16 10:10:52 UTC
If the category names are long they can overlap each other or require a third text line.



Reproducible: Always

Steps to Reproduce:
1. Open discorver using the german translation


Actual Results:  
The first three categories overlap each other.
See attached Screenshot: bug.png

Expected Results:  
No overlap
See attached Screenshot: fix.png

The attached patch solves this issue by increasing the grid-cell-width to match its content.
Comment 1 Bernhard Scheirle 2016-07-16 10:11:13 UTC
Created attachment 100124 [details]
Screenshot of the bug
Comment 2 Bernhard Scheirle 2016-07-16 10:11:51 UTC
Created attachment 100125 [details]
Screenshot of the fix
Comment 3 Bernhard Scheirle 2016-07-16 10:12:57 UTC
Created attachment 100126 [details]
Patch
Comment 4 Aleix Pol 2016-07-21 10:27:38 UTC
Hi!
Thanks for looking into the issue. :)
Could you submit the patch to phabricator.kde.org so we can discuss it?
Comment 5 Bernhard Scheirle 2016-07-21 11:08:43 UTC
>  Could you submit the patch to phabricator.kde.org so we can discuss it?

Done: https://phabricator.kde.org/D2244
Comment 7 Aleix Pol 2016-08-29 00:23:11 UTC
*** Bug 367939 has been marked as a duplicate of this bug. ***