Attendee Dialog: Focus mismatch when moving in attendee list

NEW
Unassigned

Status

Calendar
Dialogs
4 years ago
a year ago

People

(Reporter: MakeMyDay, Unassigned)

Tracking

({good-first-bug})

Lightning 3.3
good-first-bug

Details

(Whiteboard: [invite attendees dialog])

(Reporter)

Description

4 years ago
If you move through the attendee list in attendee dialog, the field, which has the cursor may differ from the highlighted one. I guess this is related to the special treatment for the (disabled) organizer entry - it cannot get the cursor but gets highlighted.

STR:
1. Create an event with some attendees
2. close and reopen the event dialog
3. open attendee dialog
4. press arrow down and up serveral times

another thing, I observed: you can genarte endlessly new rows by continously pressing the arrow down or tab key. This should be limited to one row without an entry. While this can be limited easily for the arrow down case inside the handler, the same approach for tab wouldn't work.
(Reporter)

Updated

4 years ago
Whiteboard: [invite attendees dialog]
(Reporter)

Updated

2 years ago
Whiteboard: [invite attendees dialog] → [invite attendees dialog][good first bug]
Hi,
I am new to Calendar and i would like to fix this.
Would you guide me to the files,code etc where i need to make changes.
Thanks :D
Assignee: nobody → pavankarthikboddeda
Hi Pavan,

I'm not quite sure what it takes to fix this bug, but MakeMyDay seems to have a good idea and can help you with the details. If you want to set up a development environment, I'd suggest to look at https://developer.mozilla.org/en-US/docs/Simple_Thunderbird_build. You can use http://dxr.mozilla.org/comm-central/ for code search. The most important modifier is path:foo which searches for filenames including foo. For example, you can find the attendee dialog by searching for "path:attendee" and checking the files in the calendar/ directory.

You can find me on irc.mozilla.org in #calendar if you have questions, my nickname is Fallen.
Flags: needinfo?(makemyday)
Keywords: good-first-bug
Whiteboard: [invite attendees dialog][good first bug] → [invite attendees dialog]
(Reporter)

Comment 3

2 years ago
In [1] where using setFocus at several places. It seems we're not setting the focus on the correct node in all cases. Following the STR, you may end up in a situation where the backgroundcolor highlighting is not applied to the the field where the cursor is.

When trying to reproduce, make sure you have an email identity assigned to the calendar (in the respective calendar properties in calendar view) wherein you are creating the event.

To dig into the issue, you should start with understanding how the code in the mentioned file is working.

[1] https://dxr.mozilla.org/comm-central/source/calendar/base/content/dialogs/calendar-event-dialog-attendees.xml
Flags: needinfo?(makemyday)
I am sorry , Im busy with other bugs , I could work on this bug no more. :?
Assignee: pavankarthikboddeda → nobody
You need to log in before you can comment on or make changes to this bug.