Closed
Bug 322272
Opened 19 years ago
Closed 19 years ago
should hide R button until view reversing actually works.
Categories
(Calendar :: Internal Components, defect)
Calendar
Internal Components
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dmosedale, Assigned: robin.edrenius)
References
Details
Attachments
(1 file)
|
1.47 KB,
patch
|
dmosedale
:
first-review+
|
Details | Diff | Splinter Review |
|
||
Comment 1•19 years ago
|
||
What are the issues that need to get worked out for this to work? The only bug I found was bug 322124.
|
Assignee | |
Comment 2•19 years ago
|
||
This removes the R-button. I made a comment, I'm not sure it's satisfactory though.
Assignee: base → robin.edrenius
Status: NEW → ASSIGNED
Attachment #207492 -
Flags: first-review?(jminta)
|
Reporter | |
Comment 3•19 years ago
|
||
Simon: I think you're right; that might be the only one. I'd vote for taking this patch as is, and then the patch that fixes 322124 can renable the button.
|
|
Reporter | |
Comment 4•19 years ago
|
||
Comment on attachment 207492 [details] [diff] [review] comment out R-button r=dmose
Attachment #207492 -
Flags: first-review?(jminta) → first-review+
|
|
Reporter | |
Comment 5•19 years ago
|
||
Patch landed, though I tweaked the comment slightly. Thanks, Robin!
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•