Closed Bug 1137673 Opened 9 years ago Closed 9 years ago

extra divider in the options menu of new task dialog

Categories

(Calendar :: Dialogs, defect)

Lightning 4.1
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: lazybug, Assigned: lazybug)

Details

Attachments

(2 files)

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/40.0.2214.115 Safari/537.36

Steps to reproduce:

1.open calender , add new task
2. click on "Options" menu



Actual results:

you can see there is a divider in the end on the drop-down menu


Expected results:

dividers are used to separate menu items. so there shouldn't be a menu seprate in the end of a drop-down.
Attached image a.jpg β€”
Assignee: nobody → imnmfotmal
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attached patch task-sepratorV1.patch β€” β€” Splinter Review
Attachment #8570990 - Flags: review?(philipp)
Comment on attachment 8570990 [details] [diff] [review]
task-sepratorV1.patch

Review of attachment 8570990 [details] [diff] [review]:
-----------------------------------------------------------------

This is quite simple, I think you don't mind if I take over the review Philipp ;-)

r=Decathlon 

nithin, could you please attach a patch with the new reviewer and then set checkin-needed in the keywords field?
Attachment #8570990 - Flags: review?(philipp) → review+
Thanks for taking over! Believe it or not, I just had a bugzilla conflict because I also just r+'d it. That doesn't mean you shouldn't continue taking over my reviews, its much appreciated! For that reason I'm setting checkin-needed as is. I'm fine with changing the reviewer though if you prefer.
Keywords: checkin-needed
No need to change the reviewer.
It was only to not blame you in case the review was wrong :-)
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 4.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: