Bug 3107 - Height of row in message list does not reflect font size - three columns, small screen layout
Summary: Height of row in message list does not reflect font size - three columns, sma...
Status: RESOLVED FIXED
Alias: None
Product: Claws Mail (GTK 2)
Classification: Unclassified
Component: UI/Message List (show other bugs)
Version: 3.9.3
Hardware: All All
: P3 normal
Assignee: users
URL:
Depends on:
Blocks:
 
Reported: 2014-03-12 13:36 UTC by Peter Sarvas
Modified: 2014-03-17 09:29 UTC (History)
0 users

See Also:


Attachments
Height of row in "message list" is small. (50.27 KB, image/png)
2014-03-12 13:36 UTC, Peter Sarvas
no flags Details
Patch (1004 bytes, patch)
2014-03-12 13:41 UTC, Peter Sarvas
no flags Details | Diff

Description Peter Sarvas 2014-03-12 13:36:59 UTC
Created attachment 1339 [details]
Height of row in "message list" is small.

Steps to Reproduce:
 1) Set bigger font size (ex. 18).
 2) Set "Three columns" or "Small Screen" layout.

Actual Results: 
 Height of row in "message list" is small. You can see it on screenshot.

Expected Results:
 Height of row reflect font size.
Comment 1 Peter Sarvas 2014-03-12 13:40:29 UTC
Problem is that in "summaryview.c" variable "normal_height" is set before font size is applied.

I create a patch for that.
Comment 2 Peter Sarvas 2014-03-12 13:41:24 UTC
Created attachment 1340 [details]
Patch
Comment 3 Peter Sarvas 2014-03-12 13:45:22 UTC
(In reply to comment #1)
> Problem is that in "summaryview.c" variable "normal_height" is set before
> font size is applied.
> 
> I create a patch for that.

Name of variable is "normal_row_height" not "normal_height".
Comment 4 users 2014-03-12 14:07:03 UTC
Changes related to this bug have been committed.
Please check latest Git and update the bug accordingly.
You can also get the patch from:
http://git.claws-mail.org/

++ ChangeLog	2014-03-12 14:07:02.989527772 +0100
http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=6bae7230745233c8d9264ed9056e7870b9b60ddf
Merge: 479183e cc36756
Author: Colin Leroy <colin@colino.net>
Date:   Wed Mar 12 14:07:02 2014 +0100

    Merge branch 'master' of file:///home/git/claws

http://git.claws-mail.org/?p=claws.git;a=commitdiff;h=cc3675668dcfbbd20421aad9054587e47759810e
Author: Colin Leroy <colin@colino.net>
Date:   Wed Mar 12 14:06:06 2014 +0100

    Fix bug #3107, " Height of row in message list does not
    reflect font size - three columns, small screen layout"
    Patch by Peter Sarvas.

Note You need to log in before you can comment on or make changes to this bug.