Bug 243082 - KDE help center increase font size behavior
Summary: KDE help center increase font size behavior
Status: RESOLVED FIXED
Alias: None
Product: khelpcenter
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Fedora RPMs Linux
: NOR minor
Target Milestone: ---
Assignee: Cornelius Schumacher
URL:
Keywords:
: 198568 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-06-28 22:31 UTC by shawn
Modified: 2011-09-12 19:42 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.7.2


Attachments
screen shot of text extending off the page (238.20 KB, image/png)
2010-06-28 22:31 UTC, shawn
Details

Note You need to log in before you can comment on or make changes to this bug.
Description shawn 2010-06-28 22:31:08 UTC
Created attachment 48437 [details]
screen shot of text extending off the page

Version:           unspecified (using KDE 4.4.4) 
OS:                Linux

When reading documentation in the KDE help center, the increase font size button zooms the text until it is off the screen.  I would think the proper behaviour here would be to increase the font size and maintain word wrapping to not need horizontal scrolling.

Reproducible: Always

Steps to Reproduce:
Open KDE help center
Open any help document
Press increase font size until text goes off the screen horizontally

Actual Results:  
Text zooms in until horizontal scrolling is needed to see the entire line.

Expected Results:  
word wrapping is maintained to not need horizontal scrolling to read.

This is under kde-fedora version 13.  This bug also exists in fedora 11 on a different workstation.  Please feel free to contact me if any additional information is needed or if there is some reason I should be submitting this to the fedora package maintainers.
Comment 1 Christoph Feck 2010-06-28 23:48:04 UTC
I think the issue is with the header graphics scaling. It cannot wrap the graphics, so the growing header makes the document width grow, too.
Comment 2 shawn 2010-07-09 18:25:44 UTC
(In reply to comment #1)
> I think the issue is with the header graphics scaling. It cannot wrap the
> graphics, so the growing header makes the document width grow, too.

That sounds plausible, possibly finding a document without a header could confirm this.  If you know of one let me know.
Comment 3 Christoph Feck 2011-06-23 15:41:46 UTC
*** Bug 198568 has been marked as a duplicate of this bug. ***
Comment 4 Jure Repinc 2011-09-05 11:40:31 UTC
Git commit 8a7e1ae648b209d358b8891f9abb4922256b5150 by Jure Repinc.
Committed on 05/09/2011 at 13:24.
Pushed by repinc into branch 'KDE/4.7'.

Fixed KHelpCenter Font Scaling (bug 243082)

When user clicked a button to change font size the entire document was
zoomed, instead of just the font size. This meant that the text would
run past the right edge and user had to scroll horizontaly to read the
whole line. After this patch only the font size changes.

Thanks to Jonathan Ryshpan for bringing this bug to my attention.
BUG: 243082
FIXED-IN: 4.7.2
CCMAIL: jonrysh@pacbell.net
REVIEW: 102528

M  +8    -8    khelpcenter/mainwindow.cpp
M  +1    -1    khelpcenter/mainwindow.h
M  +3    -3    khelpcenter/view.cpp
M  +2    -2    khelpcenter/view.h

http://commits.kde.org/kde-runtime/8a7e1ae648b209d358b8891f9abb4922256b5150
Comment 5 Marco Martin 2011-09-12 19:42:52 UTC
Git commit 0dca52e75fac1724ac8eacac9a4871e077889d9c by Marco Martin, on behalf of Jure Repinc.
Committed on 05/09/2011 at 13:24.
Pushed by mart into branch 'active-development/4.7'.

Fixed KHelpCenter Font Scaling (bug 243082)

When user clicked a button to change font size the entire document was
zoomed, instead of just the font size. This meant that the text would
run past the right edge and user had to scroll horizontaly to read the
whole line. After this patch only the font size changes.

Thanks to Jonathan Ryshpan for bringing this bug to my attention.
BUG: 243082
FIXED-IN: 4.7.2
CCMAIL: jonrysh@pacbell.net
REVIEW: 102528

M  +8    -8    khelpcenter/mainwindow.cpp
M  +1    -1    khelpcenter/mainwindow.h
M  +3    -3    khelpcenter/view.cpp
M  +2    -2    khelpcenter/view.h

http://commits.kde.org/kde-runtime/0dca52e75fac1724ac8eacac9a4871e077889d9c