| Summary: | "What's This" shows <ul> <li> <em> tags | ||
|---|---|---|---|
| Product: | [Plasma] kwin | Reporter: | G360 <kde-bugs.9ek5t> |
| Component: | general | Assignee: | KWin default assignee <kwin-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | lueck |
| Priority: | NOR | Flags: | mgraesslin:
ReviewRequest+
|
| Version: | 5.1.95 | ||
| Target Milestone: | --- | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| URL: | https://git.reviewboard.kde.org/r/122032/ | ||
| Latest Commit: | http://commits.kde.org/kwin/0577dee236e2a8735cf25db3d858cb6c12c1fd14 | Version Fixed In: | 5.3.0 |
| Sentry Crash Report: | |||
| Attachments: |
Screenshot showing the problem.
A small patch that simply gets rid of the <ul>s and <li>s and rename's the title to "New Window Placement" |
||
|
Description
G360
2013-02-16 18:29:11 UTC
Created attachment 77369 [details]
Screenshot showing the problem.
Proposal: remove "what's this" and name the group "Where to place new windows" or so. At least the english WT is ridiculous. Created attachment 78131 [details]
A small patch that simply gets rid of the <ul>s and <li>s and rename's the title to "New Window Placement"
I decided to quickly create a patch for this bug, but this is the first time I've done this, so I haven't gone through any official channels and its untested (but the change is minor). Sorry if I've done something wrong or if I've accidentally stepped on someone's toes. I hope the patch works/helps.
Git commit 0577dee236e2a8735cf25db3d858cb6c12c1fd14 by Martin Gräßlin. Committed on 13/01/2015 at 11:47. Pushed by graesslin into branch 'master'. [kcmkwin/options] Ensure rich text is recognized as rich text The heuristic checks whether there is something which looks like a tag before the first line break. So let's use a <br> instead of a \n for the first line break. FIXED-IN: 5.3.0 REVIEW: 122032 M +1 -2 kcmkwin/kwinoptions/advanced.ui http://commits.kde.org/kwin/0577dee236e2a8735cf25db3d858cb6c12c1fd14 |