Bug 442174 - Do not highlight or underline date (or any) links on printed reports
Summary: Do not highlight or underline date (or any) links on printed reports
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: reports (show other bugs)
Version: 5.1.2
Platform: Microsoft Windows Microsoft Windows
: NOR wishlist
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-09-08 13:21 UTC by Adilson Urso
Modified: 2024-12-20 20:11 UTC (History)
2 users (show)

See Also:
Latest Commit: https://invent.kde.org/office/kmymoney/commit/b4f5768008bfbf879bef15051949bd5eac45d6c4
Version Fixed In: 5.1.3
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Adilson Urso 2021-09-08 13:21:34 UTC 
SUMMARY

Create a option to deactivate the date links on reports. It is strange a printed report show the dates as links (underlined and blue).


STEPS TO REPRODUCE
1. 
2. 
3. 

OBSERVED RESULT


EXPECTED RESULT


SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 Dawid Wróbel 2021-09-08 14:22:50 UTC 
Just wondering why is it even an option? Why wouldn’t the links be always enabled? This seems like an unnecessary complexity which, as exemplified, is ripe for bugs.
Comment 2 Jack 2021-09-08 14:34:41 UTC 
If the report is HTML, then the underlining of links is (I believe) handled in css, so it should be possible to add a separate bit of css for printing which does not emphasize links in any way.  Links are irrelevant (and possibly distracting) in printed output.  There should be no reason to inactivate them (since they are not really active anyway) only to avoid the underlining.
Comment 3 Dawid Wróbel 2021-09-08 15:20:38 UTC 
Jack, that's exactly the reason and solution I thought of.
Comment 4 Dawid Wróbel 2021-09-08 15:23:45 UTC 
I just realized I misread the original comment – I thought that option already existed.

Adilson, adding an option just for this particular use case is a no-go. It needs to be sorted out so that the underlining is not printed without user having to do anything.
Comment 5 Jack 2021-09-08 15:28:49 UTC 
I've altered the title of the bug to be more in line with with the actual issue.

Adilson - do let us know if we have actually misunderstood your intent.
Comment 6 Adilson Urso 2021-09-08 16:36:38 UTC 
You've undestood my request
Comment 7 Adilson Urso 2021-09-09 14:41:05 UTC 
And if it is possible on exported reports too.
Comment 8 Thomas Baumgart 2021-10-23 09:35:01 UTC 
Git commit 185e333b784425f34319941b9d2115becaaa06ad by Thomas Baumgart.
Committed on 23/10/2021 at 09:34.
Pushed by tbaumgart into branch 'master'.

Don't show links in printed reports

It requires to select the KMyMoney CSS file in the reports plugin.

M  +3    -0    kmymoney/html/kmymoney.css

https://invent.kde.org/office/kmymoney/commit/185e333b784425f34319941b9d2115becaaa06ad
Comment 9 Thomas Baumgart 2021-10-23 17:24:25 UTC 
Git commit b4f5768008bfbf879bef15051949bd5eac45d6c4 by Thomas Baumgart.
Committed on 23/10/2021 at 17:23.
Pushed by tbaumgart into branch '5.1'.

Don't show links in printed reports

It requires to select the KMyMoney CSS file in the reports plugin.
(cherry picked from commit 185e333b784425f34319941b9d2115becaaa06ad)

M  +3    -0    kmymoney/html/kmymoney.css

https://invent.kde.org/office/kmymoney/commit/b4f5768008bfbf879bef15051949bd5eac45d6c4