Closed Bug 1332867 Opened 4 years ago Closed 4 years ago

Date/time picker dropmarker is collapsed under recent GTK versions (~ > 3.18 ?)

Categories

(Thunderbird :: Theme, defect)

Unspecified
Linux
defect
Not set
normal

Tracking

(thunderbird_esr5254+ fixed, thunderbird53 wontfix, thunderbird54 fixed, thunderbird55 fixed)

RESOLVED FIXED
Thunderbird 55.0
Tracking Status
thunderbird_esr52 54+ fixed
thunderbird53 --- wontfix
thunderbird54 --- fixed
thunderbird55 --- fixed

People

(Reporter: ronan, Assigned: Paenglab)

References

(Blocks 1 open bug)

Details

Attachments

(4 files, 1 obsolete file)

Since a few GTK releases (I think it was 3.18 that introduced the issue), the dropmarkers at the right of the date/time pickers in the new/edit Event dialog are collapsed (into hard-to-click zones of a few pixels wide, see attached screenshot).

Using DOM Inspector, I was able to botch a workaround with this ugly userChrome.css:

    datetimepicker * .menulist-dropmarker {width: 30px;}

To reproduce the issue and implement a proper fix, I guess booting in a VM a livecd for a distro shipping a recent GTK should do the job. Arch, or maybe the latest Fedora?

Configuration
=============

- GTK 3.22.7-1 under Arch Linux
- Standard unmodified GNOME "Adwaita" theme
- Thunderbird 51.0b2 from https://ftp.mozilla.org/pub/thunderbird/releases/51.0b2/linux-x86_64/en-US/thunderbird-51.0b2.tar.bz2
- Lightning 5.3b2 from https://ftp.mozilla.org/pub/calendar/lightning/candidates/5.3b2-candidates/build1/linux-x86_64/lightning-5.3b2.en-US.linux-x86_64.xpi
Richard, would it be possible for you to check if this is a problem with the Lightning default theme?
Flags: needinfo?(richard.marti)
I see this too in Mail Account Setup -> not a calendar only problem. It's a widget issue but the problem is, m-c uses except in tests no editable menulists and with the old CentOS builders this issue will never be seen in tests.

It's easier to fix this in TB theme as it's very unlikely it will be fixed in toolkit. I'm on a solution (similar to the proposal in comment 0) which doesn't regress on older GTK versions.
Status: UNCONFIRMED → NEW
Component: Dialogs → Theme
Ever confirmed: true
Flags: needinfo?(richard.marti)
Product: Calendar → Thunderbird
Version: Lightning 5.3 → unspecified
See Also: → 1331208
Attached patch GTK3dropmarker.patch (obsolete) — Splinter Review
This patch shows the dropmarker in Mail Account Setup and datetimepicker but does not regress on older GTK3 versions. Tested on Linux Mint with older GTK3 and Fedora with newer GTK (where no dropmarker was shown before the patch).
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8830875 - Flags: review?(acelists)
(In reply to Richard Marti (:Paenglab) from comment #3)
> This patch shows the dropmarker in Mail Account Setup and datetimepicker but
> does not regress on older GTK3 versions. Tested on Linux Mint with older
> GTK3 and Fedora with newer GTK (where no dropmarker was shown before the
> patch).

Pasted the patch in my userChrome.css for testing, looks good on GTK 3.22.7 !
Thanks for the fast fix :) , feel free to RESOLVE.
Blocks: TB52found
Based on comment 2, would there be a chance to fix this properly in toolkit?
Flags: needinfo?(enndeakin)
We can, but it would be better to find out why this was broken. Are we certain that this is a change in GTK3.18?
Flags: needinfo?(enndeakin)
I can't say it's since GTK3.18 but on Kubuntu 16.10 with GTK3.20 the dropmarker of editable menulists has only a width of 2px.
I do not see the problem on GTK3.18.9, neither in Lightning (5.6a1) nor in new account wizard (TB54).

Ronan, can you retest with current TB? Maybe the issue was already fixed on trunk.
Flags: needinfo?(ronan)
(In reply to :aceman from comment #8)
> I do not see the problem on GTK3.18.9, neither in Lightning (5.6a1) nor in
> new account wizard (TB54).
> 
> Ronan, can you retest with current TB? Maybe the issue was already fixed on
> trunk.

Testing Thunderbird Daily 54.0a1 (2017-02-12) (64-bit) with a fresh profile, the issue is still present in the New Event dialog. See attached screenshot
Flags: needinfo?(ronan)
And you are on GTK 3.22. Paenglab says it exists in GTK 3.20 already.
So can we wait until I get GTK newer than 3.18 so we see whether it isn't Ubuntu specific?
It's not Ubuntu specific, I see this on Fedora 25 too.
My distro is testing GTK 3.22 now so I should be able to check this soon.
Comment on attachment 8830875 [details] [diff] [review]
GTK3dropmarker.patch

Review of attachment 8830875 [details] [diff] [review]:
-----------------------------------------------------------------

With Gtk 3.22 this patch is needed and fixes the problem. Can you also fix it in the account creation wizard? File -> New -> Existing mail account .
Thanks
Attachment #8830875 - Flags: review?(acelists) → review+
I know you wanted to fix it TB wide with putting it in messenges.css, but maybe that dialog does not include it.
Blocks: 1360850
Added the rule in accountCreation.css too.
Attachment #8830875 - Attachment is obsolete: true
Attachment #8863158 - Flags: review+
Keywords: checkin-needed
This will need uplift as users will run TB52 on the new GTK for the years to come.
[Approval Request Comment]
User impact if declined: No visible dropmarker with newer GTK versions
Testing completed (on c-c, etc.): ready for c-c
Risk to taking this patch (and alternatives if risky): low
Attachment #8863159 - Flags: review+
Attachment #8863159 - Flags: approval-comm-esr52?
Attachment #8863159 - Flags: approval-comm-beta?
https://hg.mozilla.org/comm-central/rev/66f479be94c775dabf1da9f595b99f280c84151a
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 55.0
Attachment #8863159 - Flags: approval-comm-beta? → approval-comm-beta+
Duplicate of this bug: 1365747
Attachment #8863159 - Flags: approval-comm-esr52? → approval-comm-esr52+
Duplicate of this bug: 1371238
Duplicate of this bug: 1373584
You need to log in before you can comment on or make changes to this bug.