719197 - Rename "Reload Remote Calendar" to "Synchronize Calendars"

Status: RESOLVED FIXED

(Calendar :: Calendar Frontend, defect)

Description

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

With tabs on top we now have a "Synchronzize" toolbar button. Since the reload remote calendars button really just does a reload of everything, I think we should put these in line.

This should be seen as part of the tabs on top patch, I will likely land this on aurora too.

Comment 1

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Fix - v1

I've taken the liberty to also include a change for filter.py to ignore sunbird strings, since there was also a reload remote calendars string in there.

Comment 2

[Stefan Sitter [:ssitter]]

If "the reload remote calendars button really just does a reload of everything" as you said than "Reload" is a way better name than "Synchronize" and should stay.

Comment 3

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Since we already have an icon for synchronize I'd like to avoid renaming it to reload now. Although you will rightfully say that we haven't done this yet, the goal is to move towards a fast and stable enough storage provider so that the cache can be default, which would make synchronize correct again.

Comment 4

[Richard Marti (:Paenglab)]

Comment on attachment 589622 [details] [diff] [review]
Fix - v1

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

Looks good. I haven't checked if filter.py works but trust your skills ;)

-<!ENTITY calendar.reloadserver.label                       "Reload Remote Calendars">
-<!ENTITY calendar.reloadserver.key                         "R">
-<!ENTITY calendar.reloadserver.accesskey                   "R">
+<!ENTITY calendar.synccalendars.label                       "Synchronize Calendars">
+<!ENTITY calendar.synccalendars.key                         "R">
+<!ENTITY calendar.synccalendars.accesskey                   "S">

This three strings have a space too much so they are not well aligned.

r+ with this nit fixed.

Comment 5

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

comm-central changeset 120cf812400a
releases/comm-aurora changeset e8df1f3d2202

Comment 6

[Martin Schröder [:mschroeder]]

(In reply to Philipp Kewisch [:Fallen] from comment #5)
> comm-central changeset 120cf812400a
> releases/comm-aurora changeset e8df1f3d2202

Can this bug report be marked reolved?

Comment 7

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Yes, looks like I just forgot to close it :)