Most of entities calendar.context.delete* are useless

RESOLVED FIXED in 1.0b3

Status

Calendar
General
--
minor
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: Hubert Gajewski, Assigned: Fallen)

Tracking

Bug Flags:
blocking-calendar1.0 +

Details

(Whiteboard: [not needed beta][has l10n impact], URL)

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
Entities:
<!ENTITY calendar.context.deletethisevent.label "This Event">
<!ENTITY calendar.context.deletethisevent.accesskey "T">
<!ENTITY calendar.context.deletecompleteseries.label "Complete Series">
<!ENTITY calendar.context.deletecompleteseries.accesskey "S">
<!ENTITY calendar.context.deletefutureevents.label "All Future Events">
<!ENTITY calendar.context.deletefutureevents.accesskey "F">
<!ENTITY calendar.context.deletefuturetasks.label "All Future Tasks">
<!ENTITY calendar.context.deletefuturetasks.accesskey "F"> 
are useless.

I am not able to find their in the Lighting interface.

http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/calendar/locales/en-US/chrome/calendar/calendar.dtd&rev=1.191&mark=247#247

Comment 1

8 years ago
They have been added in Bug 374464 and it seems they were added for Bug 340025. The proposal in Bug 340025 <https://wiki.mozilla.org/Calendar:Context_Menu_Specification> still lists them. But I don't know if this bug will ever got resolved, (i.e. the entities would be used)
(Reporter)

Comment 2

8 years ago
I do not see a reason to keep these entities today. Alternatively they should be added (in the future) as a part of *complete* patch in mentioned bug.
(Assignee)

Comment 3

8 years ago
I agree, we should remove them until the final patch is ready. I doubt this will happen in the near future. We'll have to do this after string freeze though.
Flags: blocking-calendar1.0+
Whiteboard: [not needed beta][has l10n impact]
(Assignee)

Comment 4

7 years ago
Created attachment 513602 [details] [diff] [review]
Fix - v1

Plain and simple :)
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #513602 - Flags: review?(bv1578)
(Assignee)

Updated

7 years ago
OS: Mac OS X → All
Hardware: x86 → All

Comment 5

7 years ago
Comment on attachment 513602 [details] [diff] [review]
Fix - v1

r+

Could you also check the following entities from the same file? They are less "exotic" than the others but it seems they doesn't have related elements in the interface, unless the strings are build somehow in the code. Anyway I don't know if they should be deleted:

calendar.context.selectall.label
calendar.context.selectall.accesskey
calendar.context.modifytask.label
calendar.context.modifytask.accesskey
calendar.context.modifyitem.label
calendar.context.modifyitem.accesskey
calendar.context.deleteitem.label
calendar.context.deleteitem.accesskey
Attachment #513602 - Flags: review?(bv1578) → review+

Comment 6

7 years ago
...they *don't* have...
...are *built*...

never doing something when the brain has already gone to sleep.  :-)

Anyway, I've checked again and those entities don't have a related element in the interface and don't are composed as strings somewhere in the code.

Comment 7

7 years ago
OK. The next time a post without errors. I swear it. :-)
(Assignee)

Comment 8

7 years ago
Don't worry, I understood what you wanted and I didn't even notice the mistakes until you commented ;-)

I've checked the entities and they are not used, just as you've suggested. Very good catch! I'll remove the strings for checkin.
(Assignee)

Comment 9

7 years ago
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/27adcec5d969>
-> FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Trunk
(Assignee)

Comment 10

7 years ago
Backported to comm-1.9.2 <http://hg.mozilla.org/releases/comm-1.9.2/rev/a2c830b4603b>
a=philipp
Target Milestone: Trunk → 1.0b3
You need to log in before you can comment on or make changes to this bug.