<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jul 27, 2017 at 10:36 AM, Brian Grinstead <span dir="ltr"><<a href="mailto:bgrinstead@mozilla.com" target="_blank">bgrinstead@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On Jul 26, 2017, at 12:35 PM, Ehsan Akhgari <<a href="mailto:ehsan.akhgari@gmail.com">ehsan.akhgari@gmail.com</a>> wrote:<br>
> Unpopular opinion: I don't think this is a good idea.  The code involved in checking whether we are the default browser has had performance issues (see <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1357146" rel="noreferrer" target="_blank">https://bugzilla.mozilla.org/<wbr>show_bug.cgi?id=1357146</a>) and is in my humble opinion not so nice behavior which we've wanted to fix (<a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1143116" rel="noreferrer" target="_blank">https://bugzilla.mozilla.org/<wbr>show_bug.cgi?id=1143116</a>) for all users.  Hiding this from our developers seems like a disservice to our users, especially to the new users who we are hoping to attract to Firefox as we are working on improving the browser.<br>
<br>
</span>I agree about this for most features. But with the default browser prompt, I don't think improvements for our users align with improvements for our developers. The change that would make the prompt better for me is if it wasn't modal, because then I could ignore it. I don't have any reason to believe that would be an improvement for users, though.<br></blockquote><div><br></div><div>I think the exact same reason would make making the dialog non-modal an improvement for us holds for our users as well.  Just like us, they also like to use their browser to browse the web, and a modal dialog prevents them from doing that, and a modeless prompt can be ignored by them and is therefore an improvement.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="">
> I understand the desire to do the easy thing and put a band-aid on where we are bleeding for now, but perhaps we should at least consider this as a reminder to check back with the right folks working on the original effort to fix this properly for all users?<br>
<br>
</span>Maybe I’m misreading the bug, but we already don’t show the prompt on the first run (via the browser.shell.<wbr>skipDefaultBrowserCheckOnFirst<wbr>Run pref). I see the prompt on subsequent runs: so `./mach run --profile /tmp/foo` (no prompt), close browser, `./mach run --profile /tmp/foo` (prompt).<br>
<span class="HOEnZb"></span></blockquote></div><br></div><div class="gmail_extra">No, I think your understanding of the bug is correct.  We have since changed things to show the prompt on the second run I think, but the modality of it is still as hostile to the user as before.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Also, I prefixed my previous comment with "unpopular opinion" for a reason!  I wasn't really expecting anyone to resist itching this itch, because with my developer hat on I don't like to deal with this any more than anyone else here.  But I thought it was worth mentioning nonetheless, even if an objection doomed to be ignored.  :-)<br clear="all"></div><div class="gmail_extra"><br></div><div class="gmail_extra">Cheers,<br></div><div class="gmail_extra">-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Ehsan<br></div></div>
</div></div>