Closed Bug 286959 Opened 20 years ago Closed 20 years ago

make new calendar wizard suck less

Categories

(Calendar :: Sunbird Only, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mvl, Assigned: mvl)

Details

Attachments

(1 file, 1 obsolete file)

The current wizard to create a new calendar isn't the best bit of UI. It isn't
even a wizard. Just one page. This could be improved.
Attached patch checkpoint 1 (obsolete) — — Splinter Review
This is what I currently have. mostly works, but i can't test the details at
the moment. Uploading just to make sure it is safe somewhere.
From an IRC discussion with mvl:

<sipaq> seems to work basically
<sipaq> but if you don't put anything into the uri field, it just accepts that
<sipaq> there should be some checks in place
<sipaq> also 'uri' is a pretty useless field description for a normal user
<sipaq> also the wizard accepts nameless calendars, which it shouldn't
<sipaq> the wizard doesn't fully work in local-calendar-mode
<sipaq> if you press next on the second screen after choosing a name and a color
nothing happens
<sipaq> it works in remote-calendar-mode
<sipaq> but not in local-calendar-mode
<sipaq> also, your version is not localizable at the moment, but I guess this in
on purpose?
Attached patch patch v1 — — Splinter Review
This patch fixes the issues raised by sipaq, except the problem with creating
local calendars. That should be fixed by vlad, in the storage code. Also, this
isn't localizable yet, but i want review on the wizard first. Then i'll fix
that.
Attachment #178070 - Attachment is obsolete: true
Attachment #178613 - Flags: first-review?(pavlov)
Attachment #178613 - Flags: first-review?(pavlov) → first-review+
patch checked in
-> fixed. New improvements can happen in new bugs.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
I filed bug 290228 for the localizing part.
QA Contact: gurganbl → sunbird
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: