871258 - Missing privacy icon for all-day events in week-view

Status: RESOLVED FIXED

(Calendar :: Calendar Frontend, defect)

Description

[Decathlon]

Steps to reproduce:

1. in week-view create an all-day event;
2. set the privacy status to "Private Event" or to "Show Time and Date only". 

Result:
the event box doesn't show the icon related to privacy status (the icon related to the reminder is correctly showed).

Comment 1

[Decathlon]

Attachment: patch - v1

I had to add the icon directly in the "calendar-editable-item" binding.
In this way the icon appears also in the today-pane so I added a rule in order to delete it (unless we want to keep it for all-day events in the today-pane too).

Comment 2

[Philipp Kewisch [:Fallen][📱📆]]

Comment on attachment 748526 [details] [diff] [review]
patch - v1

Personally I think we should not show it in the today pane as you did in the patch, as the goal for the today pane is to give an overview of what is upcoming. The privacy status is not so important there.

r=philipp

Comment 3

[Decathlon]

(In reply to Philipp Kewisch [:Fallen] from comment #2)
> ... The privacy status is not so important there.

Ok.

Pushed on comm-central:
https://hg.mozilla.org/comm-central/rev/385e5d45032a

-> Fixed