After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 517188 - not all characters shown / missing scrollbar
not all characters shown / missing scrollbar
Status: RESOLVED FIXED
Product: gucharmap
Classification: Core
Component: general
2.21.x
Other Linux
: Normal normal
: ---
Assigned To: gucharmap maintainers
gucharmap maintainers
Depends on:
Blocks:
 
 
Reported: 2008-02-18 10:56 UTC by Jared Moore
Modified: 2008-03-31 11:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to svn trunk (1.34 KB, patch)
2008-03-06 08:41 UTC, Jared Moore
accepted-commit_now Details | Review
updated to HEAD (1.30 KB, patch)
2008-03-23 15:37 UTC, Jared Moore
none Details | Review
fixed stupid compile error (1.29 KB, patch)
2008-03-23 15:54 UTC, Jared Moore
none Details | Review

Description Jared Moore 2008-02-18 10:56:53 UTC
Steps to reproduce:

1) Open gucharmap, select a fairly small script/block (for example, Basic Latin) and reduce the font size or increase the window size until all the characters in that script/block are visible without scrolling. The scroll bar should disappear.
2) Select a character near the bottom (for example, one of the lowercase letters in Basic Latin)
3) Close gucharmap
4) Re-open gucharmap. Your selected character will be in the top row. But the scrollbar is missing, so you can't see the characters above the selected character without using the keyboard to move the selection.

Apologies if that description doesn't make sense... Just try it out and you'll see the problem :)
Comment 1 Jared Moore 2008-03-06 07:35:31 UTC
Same bug (I think):

1) Open gucharmap, select a fairly small script/block (for example, Basic
Latin)
2) Select a character near the bottom
3) Start reducing font size
4) Observe the scroll bar going nuts, and the selected character stays right at the top, even if more rows could be displayed above it
5) Click on the scrollbar, and then the scrollbar and character table will be displayed correctly.
Comment 2 Jared Moore 2008-03-06 08:41:21 UTC
Created attachment 106665 [details] [review]
patch to svn trunk

Attached patch seems to work without any obvious regressions.
Comment 3 Christian Persch 2008-03-06 14:58:15 UTC
Looks ok to me.
Comment 4 Christian Persch 2008-03-17 22:18:45 UTC
+    if (new_first_cell < 0) new_first_cell = 0;

Split that into on 2 lines please.

ok to commit to trunk and (if it applies there) to 2-22.
Comment 5 Jared Moore 2008-03-23 15:37:27 UTC
Created attachment 107874 [details] [review]
updated to HEAD
Comment 6 Jared Moore 2008-03-23 15:54:37 UTC
Created attachment 107876 [details] [review]
fixed stupid compile error

Grr... I thought I checked this before uploading. Apologies.
Comment 7 Christian Persch 2008-03-31 11:43:01 UTC
Fixed in svn trunk; thanks for the patch!