Cannot toggle read-only calendar status from calendar list anymore

NEW
Unassigned

Status

--
enhancement
10 years ago
6 years ago

People

(Reporter: ssitter, Unassigned)

Tracking

({regression})

unspecified
x86
Windows XP
regression
Bug Flags:
blocking-calendar0.9 -

Details

(Reporter)

Description

10 years ago
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.18pre) Gecko/2008090109 Calendar/0.9pre

Steps to Reproduce:
1. Start Sunbird with new profile
2. Switch to the Calendars tab in the left-handed sidebar
3. Click the icon that shows the calendar color to toggle the read-only status

Actual Results:
The read-only status of the calendar doesn't change.

Expected Results:
The read-only status of the calendar can be changed.

Regression range: Works in Sunbird 0.9pre (2008-08-31-18)
                  Fails in Sunbird 0.9pre (2008-09-01-09)
(Reporter)

Updated

10 years ago
Keywords: regression
Flags: blocking-calendar0.9?
(Reporter)

Comment 1

10 years ago
Doesn't work by clicking the new tiny lock icon either.

Updated

10 years ago
Flags: blocking-calendar0.9? → blocking-calendar0.9+

Comment 2

10 years ago
Please consider that we removed this feature intentionally:
See bug 392561 comment #22

>When discussing the problem over we noticed the following implications.
>The icons should not be displayed over a colored background because they might
>under certain colors not being distinguishable. That's why we put the icons in
>a fourth tree column on the right hand side of the calendar list. So far the
>column we use is a "cycler column" where the user can toggle the readonly state
>by clicking on it. We abandoned this functionality as it is no longer
>perceivable in our new column that is in most of the times empty.

Comment 3

10 years ago
OK, sounds reasonable and is approved by Christian.
Flags: blocking-calendar0.9+ → blocking-calendar0.9-
(Reporter)

Comment 4

10 years ago
(In reply to comment #2)
> Please consider that we removed this feature intentionally:
> See bug 392561 comment #22

Is this a final solution (i.e. this bug is WONTFIX) or a temporary solution that was just chosen to get the other bug fixed before 0.9 release? If not I'd request to add back the feature.

Comment 5

10 years ago
I think it would make sense to toggle read-only clicking the padlock.
Severity: normal → enhancement
The problem with making the readonly click come back is the column itself. There is one column for the warning icon, an empty space and the readonly lock:

* Empty space:  NOT OK: The user might accidentally click on the space and the 
                        calendar turns readonly.

* Lock icon:        OK: It feels natural to unlock by clicking on the lock icon

* Warning icon: NOT OK: Its not even obvious if the calendar is readonly or not.


If we add a new column for the warning icon, we'll be using up space we won't need in most cases.

Therefore I don't see an alternative other than WONTFIXing this bug. I have added code for my ICS inspector extension that makes this work nevertheless (since its obviously something that developers are OK with), remind me to upload a new version soon.

Stefan, what do you think? Is it ok like this? Do you have another suggestion on how we could fix it?
You need to log in before you can comment on or make changes to this bug.