Open Bug 1051572 Opened 10 years ago Updated 2 years ago

Attendee Dialog: Focus mismatch when moving in attendee list

Categories

(Calendar :: Dialogs, defect)

Lightning 3.3
defect

Tracking

(Not tracked)

People

(Reporter: MakeMyDay, Unassigned)

Details

(Keywords: good-first-bug, Whiteboard: [invite attendees dialog])

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.
Whiteboard: [invite attendees dialog]
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]
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

Hello, is this issue still open? I would like to work on this, can you guide me what files I must edit?

Hi Claire, the best thing to do is first try to reproduce the issue with a nightly build from http://ftp.stage.mozaws.net/pub/thunderbird/nightly/latest-comm-central/

Then, please check comment 3 for information on where to get started. There is also some info in comment 2. Let us know if you have specific questions.

Hi, is this ticket still open? My team for a course wishes to work on this as one of our first issues.

Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.