After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 778373 - [PATCH] quick-add-popover: focus entry after calendar selection
[PATCH] quick-add-popover: focus entry after calendar selection
Status: RESOLVED FIXED
Product: gnome-calendar
Classification: Applications
Component: User Interface
unspecified
Other Linux
: Normal normal
: 3.26
Assigned To: GNOME Calendar maintainers
GNOME Calendar maintainers
Depends on:
Blocks:
 
 
Reported: 2017-02-09 02:36 UTC by Mohammed Sadiq
Modified: 2017-02-09 11:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
quick-add-popover: focus entry after calendar selection (1.09 KB, patch)
2017-02-09 02:36 UTC, Mohammed Sadiq
committed Details | Review

Description Mohammed Sadiq 2017-02-09 02:36:46 UTC
Created attachment 345283 [details] [review]
quick-add-popover: focus entry after calendar selection

Pressing 'Enter' key shall always save the event.
So focusing on 'Add' button after selecting calendar is redundant.

Instead, focus on the entry, so that the user can add more text without
manually focusing.
Comment 1 Georges Basile Stavracas Neto 2017-02-09 11:31:06 UTC
Review of attachment 345283 [details] [review]:

I think we can do even better and check if there's anything typed into the event title. If it's empty, we focus it. If there's a title, move the focus to the "Create" button.
Comment 2 Mohammed Sadiq 2017-02-09 11:46:14 UTC
(In reply to Georges Basile Stavracas Neto from comment #1)
 > I think we can do even better and check if there's anything typed into the
> event title. If it's empty, we focus it. If there's a title, move the focus
> to the "Create" button.

I did thought this way first. Later I wrote the way above due to the following reason: if a user presses 'Enter' key when focus is on entry, it behaves the same as clicking 'Create' button. So, if the focus is one 'Create' button the extra feature we get is the 'Space bar' key, which when pressed, shall press the 'Create' button. Though pressing 'Enter' key is a bit harder than hitting 'Space bar' I thought it isn't that worth.

Anyway, if you would say otherwise again, I shall re-write my patch.

Thanks.
Comment 3 Georges Basile Stavracas Neto 2017-02-09 11:50:37 UTC
Thanks for the patch! Pushed to master on 02/09/2017, QA ready.