[Windows XP classic theme] Mouse hovering on "Other actions" button, resize message header height in some cases

RESOLVED FIXED in Thunderbird 13.0

Status

Thunderbird
Theme
--
trivial
RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: Aureliano, Assigned: dagger.bugzilla)

Tracking

({polish, pp})

Thunderbird 13.0
x86
Windows XP
polish, pp
Bug Flags:
blocking-thunderbird3 -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [STR in comment #3])

Attachments

(5 attachments, 1 obsolete attachment)

(Reporter)

Description

8 years ago
Created attachment 409500 [details]
testcase email

It seems that happens when subject of the mail is wrap and in 2 line there is a "-".

I have my TB on Windows XP that use also lightning with today pane displayed.

In attached screenshot1 you can see as is the wrap statu of the mail testcase that I have attached: when mouse over the button, the message header pane is resize to minor height and the subject not is wrapped. When mouse over end the size is restored to original measure.

I can't provide a closely STR: but using email testcase attached in scenario descibed in screenshot1, you should reproduce (I hope) scenario descrived in screenshot2.

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.5pre) Gecko/20091031 Lightning/1.0pre Shredder/3.0pre ID:20091031032222

This bug should be block TB 3? Probably not but I set blocking? for more emphasys.
Flags: blocking-thunderbird3?
(Reporter)

Comment 1

8 years ago
Created attachment 409501 [details]
screenshot1
(Reporter)

Comment 2

8 years ago
Created attachment 409502 [details]
screenshot2

mumble: could be related to lightining?
(Reporter)

Comment 3

8 years ago
Test also in safe-mode: not issue related to lightning.

I can provide a STR:

1. open TB in safe-mode and goto view "all folders";
2. open enail testcase attached here and edit as new as save it as draft;
3. goto in draft folder where the testcase is stored and enabled message pane view;
4. drag splitter between folders pane and other section until the subject ("[Bug 474701] gloda global search on toolbar, folder display refactoring mega-bug") of testcase is wrap in 2 lines: it is important that drag end immediately when subject is wrap.
5. now overing with the mouse on button and you can see screenshot2 scenario.
(Reporter)

Comment 4

8 years ago
>It seems that happens when subject of the mail is wrap and in 2 
>line there is a "-".

This is false: the issue affect all long subject email.
Whiteboard: [STR in comment #3]

Comment 5

8 years ago
Can't reproduce (Linux).
(Reporter)

Comment 6

8 years ago
(In reply to comment #5)
> Can't reproduce (Linux).
Confirmed: can't reproduce on ubuntu 9.10... it seems only windows related!

Updated

8 years ago
Keywords: pp
Summary: Mouse overing on "Other actions" button, resize message header height in some cases → [Windows] Mouse overing on "Other actions" button, resize message header height in some cases

Comment 7

8 years ago
You're using the Classic theme, right? The size of the button border is slightly smaller in Classic than Luna/etc, and the CSS for the "other actions" button sets the border to 3px (the Luna size), and when you mouse over it, the border gets forced to be 2px by way of XUL. There's another bug that mentions this, but I can't find it.
(Reporter)

Comment 8

8 years ago
Yes it is Jim: it seems to affect only windows xp classic theme and not luna theme.
(Reporter)

Updated

8 years ago
Summary: [Windows] Mouse overing on "Other actions" button, resize message header height in some cases → [Windows XP classic theme] Mouse overing on "Other actions" button, resize message header height in some cases
Assignee: nobody → nisses.mail
seems like my area
Annoying, but not blocking.
Flags: blocking-thunderbird3? → blocking-thunderbird3-

Updated

8 years ago
Duplicate of this bug: 526646

Comment 12

8 years ago
Created attachment 410498 [details]
Screenshot showing 3-line case

I'm reposting this from bug 526646 comment #1 to have all the cases in one place. As you can see, the splitter and message move up by 2px when hovering, and the "other actions" label moves up 1px and to the right 1px. The latter would support comment #7, meaning the button border is loosing 1px when the border is shown on hovering.

In contrast to the original description here and STR in comment #3, I do *not* see this in cases where the subject line wraps, hovering over the button retains the current header-pane height and wrapping status.

I can also confirm comment #6, no issue with the gnomestripe theme on KDE 3.5.

Comment 13

8 years ago
> In attached screenshot1 you can see as is the wrap statu of the mail testcase
> that I have attached: when mouse over the button, the message header pane is
> resize to minor height and the subject not is wrapped.

I think I have an explanation for the wrapping/no-wrapping case here. The width of the subject box is limited by the button box, thus the "mega-wrap" goes into the next line. When hovering over the "other actions" button, it shrinks by 2px in width, thus allowing the subject box to expand by 2px. Coincidentally, the subject now fits into a single line and is no longer wrapped.

Aureliano, if you increase or decrease the width of your window by more than 2px you should see the wrapping either disappear or present in both states.
(Reporter)

Comment 14

8 years ago
(In reply to comment #13)
> Aureliano, if you increase or decrease the width of your window by more than
> 2px you should see the wrapping either disappear or present in both states.

Yes it is: in comment #3 I says:
> it is important that drag end immediately when subject is wrap.
You have right this "immediately" is 2 px.

Comment 15

8 years ago
Ok, now I understand your steps to reproduce better. Thus, it's all indeed caused by the button shrinking when it gets a border with the Classic theme.

To avoid that, would it be possible to put a box around the menu button and force that box to have a specific height rather than the button itself?

Comment 16

8 years ago
Width is also an issue, which will be dependent on localization, etc. One solution would be to put the current button in a deck with an invisible duplicate button behind it (always using the moused-over style), and then just center the visible button. Then we don't have to worry about border size at all.

Updated

8 years ago
Keywords: polish
Target Milestone: --- → Thunderbird 3.1a1

Comment 17

8 years ago
Things should not change their size on hover. Reason is that it would exactly cause things like this - you have the same problem on the web.
Solution is to make sure that the hover state has the exact same size as the non-hover state, e.g. by not adding or removing borders.

Updated

8 years ago
Duplicate of this bug: 528564

Comment 19

8 years ago
Experiencing the same issue here, classic theme. Can't find a relation to long subject lines.

Updated

8 years ago
Summary: [Windows XP classic theme] Mouse overing on "Other actions" button, resize message header height in some cases → [Windows XP classic theme] Mouse hovering on "Other actions" button, resize message header height in some cases
Severity: normal → trivial
(Reporter)

Updated

8 years ago
Duplicate of this bug: 539652
Duplicate of this bug: 542776
A solution is to install https://addons.mozilla.org/de/thunderbird/addon/13564 .

Comment 23

8 years ago
(In reply to comment #22)
> A solution is to install https://addons.mozilla.org/de/thunderbird/addon/13564
> .

It is not true: solution is a good working TB.

Comment 24

8 years ago
Tsk meant "workaround", not "solution". (He's not a native english speaker, so be easy with him.)

Comment 25

8 years ago
(In reply to comment #24)
> Tsk meant "workaround", not "solution". (He's not a native english speaker, so
> be easy with him.)

Thank you for explanation.
I'm not English speaker too :).

Comment 26

8 years ago
I've been battling this bug for awhile and have some additional info to contribute. The info already posted points to XP running with the classic UI -- which was the one element I couldn't pin down.

What I noticed doesn't have anything to do with the length of the message subject, but rather the number of addressees. If there's only one addressee, the message header panel shrinks when the "other actions" button is "painted in" on hover. If there is more than one addressee -- i.e. two or more addressee lines -- the problem does not occur.

Comment 27

8 years ago
(In reply to comment #22)
> A [workaround] is to install https://addons.mozilla.org/thunderbird/addon/13564

Apparently herb has solved the problem by moving the "other actions" into the header-pane toolbar with all the other buttons. While that's an idea, it would increase issues with narrow windows and the buttons running into the "from:" header filed (bug 520249). So that wouldn't work with its current position.
Mass move to the new theme component.
Component: Message Reader UI → Theme
QA Contact: message-reader → theme
(Assignee)

Comment 29

6 years ago
Created attachment 555013 [details] [diff] [review]
Use 2px border on Windows Classic

I'd say the best way to fix this is to make the button look like a button, rather than try to give it a custom appearance and then be surprised when we stuff it up.

But failing that, here's a patch that sets the border to 2px instead of 3px, preventing the button size from changing on mouse-over under :-moz-system-metric(windows-classic).
(Assignee)

Updated

6 years ago
Attachment #555013 - Flags: review?(nisses.mail)
I could not get this patch to function properly on my Win7 box. Adding -moz-appearance: none; helped some, but I suspect there are some other properties interfering here.
(Assignee)

Comment 31

6 years ago
Created attachment 590996 [details] [diff] [review]
Use 2px border on Windows Classic v2

Ah, it looks like Thunderbird uses messageHeader.css from skin/classic/aero/ on Windows 7, even when Aero is disabled. (It's possible to trick it into not doing that by running under compatibility mode: Right click the Thunderbird shortcut > Properties > Compatibility > Run in compatibility mode for Windows XP.)

This patch places the rule in the aero version of messageHeader.css too.
Attachment #555013 - Attachment is obsolete: true
Attachment #590996 - Flags: review?(nisses.mail)
Attachment #555013 - Flags: review?(nisses.mail)
Comment on attachment 590996 [details] [diff] [review]
Use 2px border on Windows Classic v2

Aha, that did the trick!
review+!
Attachment #590996 - Flags: review?(nisses.mail) → review+
Keywords: checkin-needed
Checked in: http://hg.mozilla.org/comm-central/rev/81005ca68455
Assignee: nisses.mail → dagger.bugzilla
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: Thunderbird 3.1a1 → Thunderbird 13.0
You need to log in before you can comment on or make changes to this bug.