Bug 152350 - "Select font" dialog window, size list not updated
Summary: "Select font" dialog window, size list not updated
Status: RESOLVED FIXED
Alias: None
Product: kdelibs
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-11-15 12:21 UTC by FiNeX
Modified: 2007-11-16 22:04 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
"font select" dialog size update problem (121.17 KB, image/jpeg)
2007-11-15 12:21 UTC, FiNeX
Details
refresh list when moving slider (596 bytes, text/plain)
2007-11-15 14:18 UTC, Anne-Marie Mahfouf
Details

Note You need to log in before you can comment on or make changes to this bug.
Description FiNeX 2007-11-15 12:21:05 UTC
Version:            (using KDE Devel)
Installed from:    Compiled sources
OS:                Linux

The size list in the "select font" dialog window is not updated when the slider value is updated. I'll attach a screenshot.
Comment 1 FiNeX 2007-11-15 12:21:43 UTC
Created attachment 22073 [details]
"font select" dialog size update problem
Comment 2 Anne-Marie Mahfouf 2007-11-15 14:15:58 UTC
Possible patch attached. It only refreshes the list when the slider arrives at a number which is in the list. 
If the slider is set to 31, as 31 is not in the list, nothing happens in the list. 

Not sure this is really a bug, KDE 3 font chooser already had this behavior that the intbox did not refresh the list.

Comment 3 Anne-Marie Mahfouf 2007-11-15 14:18:42 UTC
Created attachment 22079 [details]
refresh list when moving slider
Comment 4 David Faure 2007-11-15 14:30:53 UTC
Why MatchContains? I can't think of a bug that this would trigger, assuming the size list is sorted,
but still a MatchExactly would be simpler, no?
Comment 5 FiNeX 2007-11-15 14:44:54 UTC
The patch is working, I've just tried. If there is a simpler way, It will be better :-)
Comment 6 Anne-Marie Mahfouf 2007-11-16 22:04:01 UTC
SVN commit 737652 by annma:

synch fonts list with slider
reviewed by dfaure, thanks
BUG=152350


 M  +6 -0      kfontchooser.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=737652