Calendar tab toolbar buttons are missing tooltips

RESOLVED FIXED in 4.3

Status

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: aleth, Assigned: ssitter)

Tracking

Trunk

Details

Attachments

(1 attachment, 1 obsolete attachment)

For a new user, it would be helpful if the calendar tab buttons (Synchronize, Event, Task,...) had explanatory tooltips which explain what the buttons are for (e.g. synchronize with what? does it open an Event view or add a new event or...?)
Flags: tb-integration?
This is pretty easy, taking :)
Assignee: nobody → ssitter
Status: NEW → ASSIGNED
Attachment #8602914 - Flags: review?(matthew.mecca)
Comment on attachment 8602914 [details] [diff] [review]
add tooltips to Calendar tab and Task tab toolbar buttons

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

Looks good, r=mmecca

::: calendar/locales/en-US/chrome/lightning/lightning-toolbar.dtd
@@ +7,5 @@
>  <!ENTITY lightning.toolbar.task.name "Task Toolbar">
>  <!ENTITY lightning.toolbar.task.name.accesskey "T">
>  
>  <!ENTITY lightning.toolbar.sync.label "Synchronize">
> +<!ENTITY lightning.toolbar.sync.tooltip "Reload calendar and synchronize changes">

We should probably use the plural form "calendars" here since all calendars are refreshed.

@@ +12,2 @@
>  <!ENTITY lightning.toolbar.delete.label "Delete">
> +<!ENTITY lightning.toolbar.delete.tooltip "Delete selected event or task">

Here also, since there could be multiple events or tasks deleted.

@@ +18,2 @@
>  <!ENTITY lightning.toolbar.print.label "Print">
> +<!ENTITY lightning.toolbar.print.tooltip "Print selected event or task">

Should this be more generic, since the print dialog isn't limited to the current selection? Maybe just "Print events and tasks"
Attachment #8602914 - Flags: review?(matthew.mecca) → review+
(In reply to Matthew Mecca [:mmecca] from comment #3)
> We should probably use the plural form "calendars" here since all calendars
> are refreshed.
> ...
> Here also, since there could be multiple events or tasks deleted.

I tried to align with tooltips used in Thunderbird and they use singular although the command applies to one or more elements (e.g. Delete selected message or folder)

> Should this be more generic, since the print dialog isn't limited to the
> current selection? Maybe just "Print events and tasks"

Sure, I can update the patch.
Attachment #8602914 - Attachment is obsolete: true
Attachment #8613023 - Flags: review?(matthew.mecca)
Comment on attachment 8613023 [details] [diff] [review]
add tooltips to Calendar tab and Task tab toolbar buttons, v2

r=mmecca
Attachment #8613023 - Flags: review?(matthew.mecca) → review+
Keywords: checkin-needed
Pushed to https://hg.mozilla.org/comm-central/rev/331e683af864
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Flags: tb-integration?
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 4.3
You need to log in before you can comment on or make changes to this bug.