Closed Bug 1303248 Opened 3 years ago Closed 3 years ago

Make modal highlighting nightly only.

Categories

(Toolkit :: Find Toolbar, defect)

51 Branch
defect
Not set

Tracking

()

VERIFIED FIXED
mozilla51
Tracking Status
firefox51 --- verified
firefox52 --- verified

People

(Reporter: arai, Assigned: arai)

References

Details

Attachments

(1 file)

https://hg.mozilla.org/mozilla-central/rev/5febcc03a729
it's enabled by default on all branches now,

it should be behind NIGHTLY_BUILD.
bug 1291278 should be fixed (or at least reduce the amount of blocking bugs) before enabling this on all branches.
Attachment #8791862 - Flags: review?(jaws)
Comment on attachment 8791862 [details] [diff] [review]
Make modal highlighting nightly only.

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

::: browser/app/profile/firefox.js
@@ +1524,5 @@
>  // crash report notification across different days and shutdown
>  // without a user choice before we suppress the notification for
>  // some number of days.
>  pref("browser.crashReports.unsubmittedCheck.chancesUntilSuppress", 4);
> +pref("browser.crashReports.unsubmittedCheck.autoSubmit", false);

I can't see what happened here... \n\r vs \n?
Attachment #8791862 - Flags: review?(jaws) → review+
Assignee: nobody → arai.unmht
Status: NEW → ASSIGNED
Thank you for reviewing :)

(In reply to Mike de Boer [:mikedeboer] from comment #2)
> Comment on attachment 8791862 [details] [diff] [review]
> Make modal highlighting nightly only.
> 
> Review of attachment 8791862 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: browser/app/profile/firefox.js
> @@ +1524,5 @@
> >  // crash report notification across different days and shutdown
> >  // without a user choice before we suppress the notification for
> >  // some number of days.
> >  pref("browser.crashReports.unsubmittedCheck.chancesUntilSuppress", 4);
> > +pref("browser.crashReports.unsubmittedCheck.autoSubmit", false);
> 
> I can't see what happened here... \n\r vs \n?

accidentally added newline at the end of file, where newline wasn't there.
reverted locally.
https://hg.mozilla.org/mozilla-central/rev/00c35bd8acb9
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Verified as fixed on 
 - Nightly 52.0a1, BuildID: 20160919065232  - modal highlighting is on
 - Aurora 51.0a2. buildID: 20160920004004 -  modal highlighting is off
Status: RESOLVED → VERIFIED
The new findbar in Firefox 50 does not keep highlight active after the bar is closed. All previous versions kept. This is being very uncomfortable for me. I would suggest an option for that, if it was desired. I would also go against "dim the page" change, it simply goes against my usability criteria in many ways.
What bug# tracks these changes to the findbar?
Depends on: 1316516
You need to log in before you can comment on or make changes to this bug.