| Summary: | No second row for header | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Christoph <schwille> |
| Component: | Notifications | Assignee: | Kai Uwe Broulik <kde> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | mklapetek, notmart |
| Priority: | NOR | ||
| Version First Reported In: | 5.8.0 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Fedora RPMs | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
| Attachments: |
Uncomplete header of notification
Website sends only header Website with correct header / body Nextcloud's notification |
||
I am a bit unsure if this really should be changed. The header is only a headline. The actual contents comes after the headline. If we allowed wrapping the headline to multiple lines, we might fill up the space and have no room to show the (important?) contents. Could you please show a screen shot of an actual "Gnotifier" message? Maybe that addon needs to be changed to move the text to the content rows, instead of showing it at the headline. Created attachment 102091 [details]
Website sends only header
It depends on the website. Some web sites send correct header/bodys, some web sites send only header.
Created attachment 102092 [details]
Website with correct header / body
the header should really be just some short title. as a workaround of things sending incorrect content, it could be done that if only header text is present, to display it in the body block instead What about a mouse hover that could show the whole text? Unfortunately I'm not able to code it... Created attachment 102317 [details]
Nextcloud's notification
I won't enable word wrapping for the summary, as this is "a single line overview of the notification", according to the spec. (The spec does say servers *should* word wrap if neccessary but I disagree, this contradicts the "single line" statement) I like Marco's idea of putting the summary as body text if only a summary is given, and then perhaps use the application name as title instead. Yes, I highly agree with the propositions made! Would be a good feature. solved by commit cbf06cf862c5779f2c16a6903e828c67d5c1a670 |
Created attachment 102053 [details] Uncomplete header of notification It doesn't appear a second row for the header of a notification when the message is too long for the widget size. Steps to reproduce: 1.) Run "notify-send -t 0 testtesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttest123 test" 2.) You can't see the whole message. "123" of the string is lost. Expected results: Instead of "..." the whole information should appear I know that a summary isn't often so long but some HTML5 notifications, forwarded through Firefox' Gnotifier addon, contain long headers instead of a short summary in the header and a long body... Would be great to read all information. Thanks!