Closed Bug 1561252 Opened 1 year ago Closed 1 year ago

In today pane, on daily build , the text is outside the screen instead of being adapted

Categories

(Calendar :: Calendar Views, defect)

Lightning 68
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aarnaud, Assigned: pmorris)

References

(Blocks 1 open bug)

Details

(Keywords: regression)

Attachments

(5 files, 1 obsolete file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Firefox/68.0

Steps to reproduce:

  1. Display the today pane
  2. Create a event with a big title like this "I do a test for Thunderbird daily, the text goes out of the screen wheread adapated to the today pane before"

Actual results:

A part of the event title is outside of the screen.
I've attach a screenshot of the result.

Expected results:

It should be adapted to the screen space.
Note: As I remember it works in last stable release (60)

Checking now,

The issue isn't reproducible on my computer in 68.0 beta1 or beta2. Are you using a custom font or theme? It could be something other than TB causing the pain to not display the title properly.

Component: Untriaged → Calendar Views
Product: Thunderbird → Calendar

I think, it's because in French the label "Prochainement (4 se..." is wider than the visible area. When the splitte would be moved to the left until on the bottom the scrollbar vanishes, then all would be good.

(In reply to Benjamin Flanagin from comment #1)

Checking now,

The issue isn't reproducible on my computer in 68.0 beta1 or beta2. Are you
using a custom font or theme? It could be something other than TB causing
the pain to not display the title properly.

Indeed, I've changed the font on my system but on TB 60 with the same steps it works. I've attach a screenshot with TB 60.

This could be a regression from the de-XBL work. But I can't say from which.

I'd guess from bug 1534382?
Some css adjustments needed.

Assignee: nobody → paul
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regression

I tried to reproduce this, but it is working for me on a current trunk build. This is on Ubuntu/GNU/Linux.

Try it with making the "Upcoming" label longer than the minimum today pane width. Then you see that it gets a scrollbar on the bottom of this list.

Version: Trunk → Lightning 7.1

FYI, I reproduce the same issue on Thunderbird 68.0b2.

Best regards,
Alex.

Version: Lightning 7.1 → Lightning 7.0

Thanks Richard. Yep, I can reproduce it that way now. Working on a fix now.

Attached patch today-pane-text-wrap-0.patch (obsolete) — Splinter Review

This patch appears to fix the problem with a couple of CSS changes (at least here on linux).

Try server run for good measure, although I wouldn't expect this to affect any tests:
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=9d376bea239140e8a32264ddb2a274d9e77d51f9

Attachment #9075403 - Flags: review?(richard.marti)

Screenshot with it working.

Status: NEW → ASSIGNED
Comment on attachment 9075403 [details] [diff] [review]
today-pane-text-wrap-0.patch

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

Thanks, this works. r=me with the comment considered.

::: calendar/base/themes/common/today-pane.css
@@ +43,5 @@
>    color: inherit;
>  }
>  
> +.today-pane-panel-vbox {
> +  width: inherit;

Is this needed? Is the flex="1" not enough?
Attachment #9075403 - Flags: review?(richard.marti) → review+

Thanks for the review.

(In reply to Richard Marti (:Paenglab) from comment #13)

.today-pane-panel-vbox {
width: inherit;

Is this needed? Is the flex="1" not enough?

Huh, I just double checked and apparently it is not needed, so I've taken this part out. (Somehow in the process of working on it, it appeared to be needed before...)

Attachment #9075403 - Attachment is obsolete: true
Keywords: checkin-needed
Attachment #9075443 - Flags: review+

jorgk: This should land on beta and trunk. This patch applies cleanly on the beta repo, so I haven't made a separate beta patch.

Attachment #9075443 - Flags: approval-calendar-beta?(philipp)

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/81b346228baa
Fix text wrapping in today pane. r=Paenglab

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 7.1
Attachment #9075443 - Flags: approval-calendar-beta?(philipp) → approval-calendar-beta+
You need to log in before you can comment on or make changes to this bug.