View Issue Details

IDProjectCategoryView StatusLast Update
0006548JEDI VCL00 JVCL Componentspublic2019-05-02 08:32
ReporterrmetzgerAssigned Tojfudickar 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version 
Target VersionFixed in Version 
Summary0006548: TJvMainMenu and TJvPopupMenu do not use TScreen.MenuFont
DescriptionHey,
TJvMainMenu (line 1258) and TJvPopupMenu (line 1839) use the font given by tagNONCLIENTMETRICS.lfMenuFont, but they should use TScreen.MenuFont instead.
BTW, both object correctly use TScreen.MenuFont for measuring menu item (line 1300 and 1881, respectively).
Workaround is simple, just replace SetDefaultMenuFont(Canvas.Font); with FCanvas.Font:=Screen.MenuFont on lines 1258 and 1839.
IMO, Function SetDefaultMenuFont is irrelevant here.
Best regards,
TagsNo tags attached.

Activities

obones

2018-07-18 15:51

administrator   ~0021517

Could you check if the issue is still present in the latest GIT content? If yes, please provide the zipped sources of an application showing this.

mh

2019-04-27 14:41

reporter   ~0021765

Pull request for this created.
https://github.com/project-jedi/jvcl/pull/101

mh

2019-04-30 21:23

reporter   ~0021779

Pull request replaced by this one:
https://github.com/project-jedi/jvcl/pull/110

jfudickar

2019-05-02 08:32

developer   ~0021789

Fixed in Repository

Issue History

Date Modified Username Field Change
2017-03-03 16:49 rmetzger New Issue
2018-07-18 15:51 obones Note Added: 0021517
2018-07-18 15:51 obones Status new => feedback
2019-04-27 14:41 mh Note Added: 0021765
2019-04-30 21:23 mh Note Added: 0021779
2019-05-02 08:32 jfudickar Note Added: 0021789
2019-05-02 08:32 jfudickar Status feedback => resolved
2019-05-02 08:32 jfudickar Resolution open => fixed
2019-05-02 08:32 jfudickar Assigned To => jfudickar