Closed Bug 1642675 Opened 4 years ago Closed 4 years ago

Make the calendar preferences look better aligned

Categories

(Calendar :: General, defect)

defect

Tracking

(thunderbird78 fixed)

RESOLVED FIXED
Thunderbird 78.0
Tracking Status
thunderbird78 --- fixed

People

(Reporter: Paenglab, Assigned: Paenglab)

Details

Attachments

(1 file, 1 obsolete file)

The calendar prefs need a more homogenous sizing of the menulists in the first block. And the categories list with its buttons could look like the tags list to be more consistent in the prefs.

Attached patch 1642675-calendar-preferences.patch (obsolete) — — Splinter Review

I'm asking aleca for review because this is more review about UI changes.

Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #9153461 - Flags: review?(alessandro)
Attachment #9153461 - Flags: approval-calendar-beta?(paul)
Comment on attachment 9153461 [details] [diff] [review]
1642675-calendar-preferences.patch

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

This is very good, thank you for taking care of it, just a CSS nit.

Can you also update the commit message with: "Improve the alignment of the sections in the Calendar Preferences tab"

::: calendar/base/content/preferences/views.inc.xhtml
@@ +3,5 @@
>  # file, you can obtain one at http://mozilla.org/MPL/2.0/.
>  
>      <html:fieldset data-category="paneCalendar">
>          <separator/>
> +        <box style="display:grid; grid-template-columns: auto 1fr 1fr;">

Let's avoid inline style as much as possible and put this in the CSS.
Attachment #9153461 - Flags: review?(alessandro) → review+

Moved the grid style to preferences.css.

Attachment #9153461 - Attachment is obsolete: true
Attachment #9153461 - Flags: approval-calendar-beta?(paul)
Attachment #9153500 - Flags: review+
Attachment #9153500 - Flags: approval-calendar-beta?(paul)
Attachment #9153500 - Flags: approval-calendar-beta?(paul) → approval-calendar-beta+
Target Milestone: --- → 79

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/42d651f7162a
Improve the alignment of the sections in the Calendar Preferences tab. r=aleca

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: 79 → 78
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: