Indicate that a comment is required when selecting a value which has an auto-comment configured

RESOLVED FIXED

Status

()

enhancement
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: sylvestre, Assigned: glob)

Tracking

Production

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

It would be nice if we could have an automatic focus to the regular bugzilla textarea when selecting relnotes? or tracking-firefoxXX? are used.

Please consider that we may want to disable this feature if it is a pain for users (I guess people don't like when the focus changes).
Summary: Autofocus on the comment textarea when selecting relnotes? → Autofocus on the comment textarea when selecting a value which has an auto-comment configured
No longer depends on: 853108
Or, maybe a better solution, instead of changing the focus, maybe open a new textarea close to tracking flag to make sure that the user sees it (without the focus change)
(In reply to Sylvestre Ledru [:sylvestre] from comment #1)
> Or, maybe a better solution, instead of changing the focus, maybe open a new
> textarea close to tracking flag to make sure that the user sees it (without
> the focus change)

i think that focusing and scrolling the textarea is a better solution than creating/moving the comment textarea.


another option which may be simpler is displaying a short notification next to the flag..

  [ relnotes-firefox? ] (comment required)

clicking on the notification scrolls the comment textarea into view and focuses is.
Yeh, you like better your proposal :)
Assignee: nobody → glob
FYI, most of the people don't see the autocomment. This gives stuff like:
https://bugzilla.mozilla.org/show_bug.cgi?id=1048099#c1
Posted patch 1041964_1.patch (obsolete) — Splinter Review
Attachment #8467583 - Flags: review?(dkl)
Comment on attachment 8467583 [details] [diff] [review]
1041964_1.patch

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

::: extensions/TrackingFlags/web/js/tracking_flags.js
@@ +52,4 @@
>          return;
> +    }
> +    // create "comment required"
> +    var span = document.createElement('span');

Need to check to see if this span is already present, otherwise if the user first selects one value with a required comment, but then selects another, it keeps adding another (comment required) each time.
Attachment #8467583 - Flags: review?(dkl) → review-
Duplicate of this bug: 1051186
Attachment #8467583 - Attachment is obsolete: true
Attachment #8470631 - Flags: review?(dkl)
Comment on attachment 8470631 [details] [diff] [review]
1041964_2.patch

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

r=dkl
Attachment #8470631 - Flags: review?(dkl) → review+
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   cf5c164..2957d62  master -> master
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Summary: Autofocus on the comment textarea when selecting a value which has an auto-comment configured → Indicate that a comment is required when selecting a value which has an auto-comment configured
You need to log in before you can comment on or make changes to this bug.