Open Bug 1325171 (photon-visual) Opened 7 years ago Updated 8 months ago

[meta] Photon - Firefox visual refresh (Theme)

Categories

(Firefox :: Theme, defect, P5)

defect

Tracking

()

People

(Reporter: canuckistani, Unassigned)

References

(Depends on 13 open bugs, Blocks 1 open bug)

Details

(Keywords: meta, Whiteboard: [photon-visual])

We're investigating a visual refresh of Firefox this year, this bug tracks that work.
Priority: -- → P3
Personally I like the look of the developer edition theme. Maybe a light version of it would look very nice and modern.
+1 for developer edition theme or something minimal like that. Dark one is great.
(In reply to Allan Gardner (:Mathnerd314) from comment #1)
> I know it was an April Fool, but I'll bring up this blog post:
> https://blog.mozilla.org/ux/2013/04/firefox-next-evolving-the-user-interface-
> using-genetic-algorithms/. I haven't seen any comprehensive browser UI
> analysis based on Fitt's Law, Hick's Law, and GOMS-CPM/CoDeIn modeling.
> Optimize toolbar button sizes for task flow. Don't just put lipstick on a
> pig, save a few seconds for millions of users! :)

(In reply to modestdev from comment #3)
> +1 for developer edition theme or something minimal like that. Dark one is
> great.

The developer edition themes are shipping as alternative themes in Firefox 53, see bug 1314091.
Alias: photon → photon-visual
No longer depends on: 1336227, 1336241
Depends on: 1341048
No longer depends on: 1336230
No longer depends on: 1337432
No longer depends on: 759252
Depends on: 1348294
Depends on: 1347543
Keywords: meta
Summary: Firefox visual refresh → [meta] Firefox visual refresh
Whiteboard: [photon]
Depends on: 1349552
Depends on: 1349555
Depends on: 1058040
No longer blocks: Quantum
Please consider for the new default theme:
* to not show the protocol or a padlock when https:// is used - encryption=neutral - (but display the protocol on all other protocols): Meta bug 1335586 (https-everything)
* to show a red crossed padlock (bug 1310447) with a red text 'Not Secure' (bug 1335970) for every unencrypted protocol (http://, ftp://, ws://)
* to show a green padlock without a text if DNSSEC/DANE stapling is valid (bug 672600, a MOSS project is working on it)
* to show a blue padlock with a blue Name for EV certificates (but green padlock + green text if DNSSEC/DANE stapling is valid)
(let EV green, as long as DNSSEC/DANE Stapling (bug 672600) is not implemented, but change it then)
Thank you :)
> * to not show the protocol or a padlock when https:// is used - encryption=neutral - (but display the protocol on all other protocols): Meta bug 1335586 (https-everything)

I think this is the general direction we're heading for eventually, but it's not within scope for the visual refresh. Which is good, because there are a lot of factors to consider for this decision. (It's also not necessary to do this as part of a major UI refactor, since the identity block is pretty isolated.)

> * to show a red crossed padlock (bug 1310447) with a red text 'Not Secure' (bug 1335970) for every unencrypted protocol (http://, ftp://, ws://)

Bug 1310447 comment 7 outlines our team's current decision on this: We will not show additional text in the URL bar, since the user will get a much more efficient insecure login warning when they try to use a login form on that page. The "Not Secure" text does not add any additional value in our opinion. We want to avoid adding unnecessary elements that clutter the URL bar (even more so with the refresh that focuses on gaining simplicity instead of adding complexity).

The bug is not closed yet, because this decision could be reconsidered in the future. For now, we're not doing it.

> * to show a green padlock without a text if DNSSEC/DANE stapling is valid (bug 672600, a MOSS project is working on it)
> * to show a blue padlock with a blue Name for EV certificates (but green padlock + green text if DNSSEC/DANE stapling is valid)
(let EV green, as long as DNSSEC/DANE Stapling (bug 672600) is not implemented, but change it then)

I generally agree that DNSSEC/DANE sounds very exciting and we should probably think about how we can support it UI-wise, but really, there is a lot to consider for this and the general UI refresh will be a big project on its own. Feel free to make new bugs in the Security component for DNSSEC/DANE UI highlighting, so that we're able to discuss this separately.

I do not mean to shut down your (very interesting) ideas, just to say that they're not in scope for the UI refresh and that changing the identity indicators is always a delicate topic :)
Summary: [meta] Firefox visual refresh → [meta] Photon - Firefox visual refresh
Firefox on Android needs a visual refresh too, perhaps with some Material Design and better integration with the Android system.
(In reply to uncertainquark from comment #8)
> Firefox on Android needs a visual refresh too, perhaps with some Material
> Design and better integration with the Android system.

This has been in discussion for a matter of years. See bug 1074220
(In reply to uncertainquark from comment #8)
> Firefox on Android needs a visual refresh too, perhaps with some Material
> Design and better integration with the Android system.

This is being worked on. No bugs filed quite yet but the goal is to have a consistent look across Firefox Browsers ( Android, iOS, Desktop )
Depends on: 1351268
No longer depends on: 1351268
For the complete UI rewrite for a non-XUL Firefox, please have these in mind (for your technical design decisions) to have it easy to implement in the future:
* theme-color meta tag support like Chrome Mobile/Vivaldi (https://bugzilla.mozilla.org/buglist.cgi?quicksearch=theme-color)
* scrollbar styles like webkit, as it wasn't possible with XUL: bug 77790

(In reply to Johann Hofmann [:johannh] from comment #7)
> Feel free to make new bugs in the Security component for DNSSEC/DANE UI highlighting, so that we're able to discuss this separately.
bug 1351684 with a suggestion image
> I do not mean to shut down your (very interesting) ideas, just to say that
> they're not in scope for the UI refresh and that changing the identity
> indicators is always a delicate topic :)
But please overthink currenty thoughts about showing the word "Secure" as people couldn't notice the diffence to EV certificates in thoughtless moments. People are already complaining about Letsencrypt certificates for phishing sites as Chrome shows the pseudo EV certificate owner named "Secure". HTTPS for a phishing site is technically a "Secure" connection, sure, but we shouldn't overvalue HTTPS. Even if phishing sites had DNSSEC/DANE Stapling at some time, it wouldn't be good to show the word "Secure". Firefox hasn't something like a click-to-play for javascript, so websites can't be "Secure". "Secure" could be an OV-validated certificate with OCSP Must staple and with strong Content-Security-Policy - at the moment. As you say:
> We want to avoid adding unnecessary elements that clutter the URL bar
Depends on: 1256754
Depends on: photon-touch
(In reply to bugzilla from comment #11)
> But please overthink currenty thoughts about showing the word "Secure" as
> people couldn't notice the diffence to EV certificates in thoughtless
> moments. People are already complaining about Letsencrypt certificates for
> phishing sites as Chrome shows the pseudo EV certificate owner named
> "Secure". HTTPS for a phishing site is technically a "Secure" connection,
> sure, but we shouldn't overvalue HTTPS. Even if phishing sites had
> DNSSEC/DANE Stapling at some time, it wouldn't be good to show the word
> "Secure". Firefox hasn't something like a click-to-play for javascript, so
> websites can't be "Secure". "Secure" could be an OV-validated certificate
> with OCSP Must staple and with strong Content-Security-Policy - at the
> moment.

You might be relieved to hear that we're not interested in showing "Secure" for HTTPS sites, mostly for the reasons that you're stating. I've seen this in some mockups that are floating around but that was early WIP and not coordinated with the security team.
No longer depends on: 1256754
Depends on: 1196266
No longer depends on: 1344917
No longer depends on: 1344910
No longer depends on: 1344907
Depends on: 1352361
Depends on: 1352366
Flags: qe-verify-
Priority: P3 → --
Depends on: 1355455
Depends on: 1173732
(In reply to marczellm from comment #13)
> Why is the visual refresh even necessary? In my user-perspective the
> Australis look just stabilized like "yesterday" when bug 734326 got fixed.
> Seems like bikeshedding to me.

Sorry, this is a tracking bug and not a discussion platform. Please take that concern to an appropriate mailing list or user forum. Note that a significant chunk of the people involved in this are Firefox peers, so it's unlikely your arguments will be able to stop the project. We do, however, appreciate ideas and constructive feedback (but please not in this specific bug).
Depends on: 1355764
Depends on: 1185482
Depends on: photon-tabs
No longer depends on: 1349552
No longer depends on: 1349555
No longer depends on: 1355764
No longer depends on: 1173732
No longer depends on: 1185482
No longer depends on: 1347543
Depends on: 1356202
No longer depends on: 1356202
Whiteboard: [photon] → [photon-visual]
(In reply to Jeff Griffiths (:canuckistani) (:⚡︎) from comment #0)
> We're investigating a visual refresh of Firefox this year, this bug tracks
> that work.

Is there a place where this is being discussed, with images?
Depends on: 1351268
Depends on: 1357858
Depends on: 1357863
No longer depends on: 1357858
No longer depends on: 1357863
unification/quantum: library in a tab (bug 697359) like chromium or the fancy about:preferences do. currently it's xul and webrender makes every window red if I close the library (downloads). Please evaluate a "everything in a tab or as bubble like the downloads panel" policy as part of Photon's style guide.
Depends on: 1361686
Depends on: 1362008
Depends on: 1362011
Depends on: 1362083
Depends on: 1362100
Sorry for bugmail spam. I'm removing recent regression bugs again, since we don't need every photon-visual bug to block this one directly. They still block this indirectly via higher-level bugs.
No longer depends on: 1362008, 1362011, 1362083, 1362100
Depends on: 1363028
Depends on: 1363046
No longer depends on: 1363046
Depends on: 1363033
Depends on: 1363056
No longer depends on: 1363033
Depends on: 1363477
Depends on: 1363485
Depends on: 1363486
Depends on: 1363487
Depends on: 1363502
Depends on: 653670
Depends on: 1204609
Depends on: 1363909
No longer depends on: 1363486
No longer depends on: 1363487
(In reply to Darkspirit from comment #16)
> unification/quantum: library in a tab (bug 697359) like chromium or the
> fancy about:preferences do. currently it's xul and webrender makes every
> window red if I close the library (downloads). Please evaluate a "everything
> in a tab or as bubble like the downloads panel" policy as part of Photon's
> style guide.

Bug 1325171 is about the Photon Theme and bug 1346488 is about Photon itself.

For "everything in a tab" please look also at bug 1364703.
Summary: [meta] Photon - Firefox visual refresh → [meta] Photon - Firefox visual refresh (Theme)
Depends on: 1365213
Depends on: 1364999
Depends on: 1365552
No longer depends on: 1365552
Depends on: 1355330
Depends on: 1355326
Depends on: 1355328
Depends on: 1365003
Depends on: 1365173
Depends on: 1365906
Depends on: 1365024
Depends on: 1366009
No longer depends on: 1366009
Depends on: 1366172
Depends on: 1366424
No longer depends on: 1366172
Depends on: 1366492
Depends on: 1366555
Depends on: 1366749
No longer depends on: 1366749
Depends on: 1367384
Depends on: 1367385
Depends on: 1367419
Depends on: 1367421
No longer depends on: 1367419
No longer depends on: 1367421
Depends on: 1367432
Depends on: 1367439
Depends on: 1367712
Depends on: 1368161
No longer depends on: 1365906
No longer depends on: 1365213
No longer depends on: 1366492
No longer depends on: 1363502
Depends on: 1370929
Depends on: 1367149
Depends on: 1372689
Depends on: 1376241
Depends on: 1376578
No longer depends on: 1376578
No longer depends on: 1379778
No longer depends on: 1372689
Depends on: 1369415
Depends on: 1383026
Depends on: 1384541
Depends on: 1384893
Depends on: 1384895
Depends on: 1384898
Depends on: 1385702
Depends on: 1387043
Depends on: 1385657
Depends on: 1387420
No longer depends on: 1376241
Depends on: 1387784
Depends on: 1387738
Depends on: 1387614
Depends on: 1387750
Depends on: 1387762
Depends on: 1387705
No longer depends on: 1388529
Depends on: 1388676
Blocks: 1369415
No longer depends on: 1369415
No longer blocks: 1369415
Depends on: 1389476
No longer depends on: 1389476
Depends on: 1389476
No longer depends on: 1389476
Depends on: 1389740
Depends on: 925284
No longer depends on: 925284
Depends on: 1390812
Depends on: 1390846
Depends on: 1390359
Depends on: 1391191
Depends on: 1391227
Depends on: 1391539
No longer depends on: 1391539
No longer depends on: 1389646
Depends on: 1392416
Depends on: 1392541
Depends on: 1391395
Depends on: 1391613
No longer depends on: 1392772
Depends on: 1386425
Depends on: 1389792
Depends on: 1393054
Depends on: 1393411
No longer depends on: 1393411
Depends on: 1390885
Depends on: 1389569
Depends on: 1389223
Depends on: 1389462
Depends on: 1393802
No longer depends on: 1393777
No longer depends on: 1394064
Depends on: 1394248
No longer depends on: 1384686
No longer depends on: 1387609
Depends on: 1395081
No longer depends on: 1392978
Depends on: 1395084
Depends on: 1395434
Depends on: 1395436
No longer depends on: 1395084
Depends on: 1394933
Depends on: 1395596
No longer depends on: 1395436
No longer depends on: 1395081
Depends on: 1397285
Depends on: 1392219
Depends on: 1394137
Depends on: 1397265
No longer depends on: 1397265
Depends on: 1397797
Depends on: 1387354
No longer depends on: 1397797
Depends on: 1398066
No longer depends on: 1398066
Depends on: 1398125
Depends on: 1398187
Depends on: 1398319
Depends on: 1398173
Depends on: 1398696
Depends on: 1398708
Depends on: 1398707
Depends on: 1398747
Depends on: 1398750
Depends on: 1398761
Depends on: 1398773
Depends on: 1399025
Depends on: 1399047
Depends on: 1399072
No longer depends on: 1398173
No longer depends on: 1398758
No longer depends on: 1398761
Depends on: 1399078
No longer depends on: 1399047
No longer depends on: 1399072
Depends on: 1399084
Depends on: 1399138
Depends on: 1399383
Depends on: 1397761
Depends on: 1398332
Depends on: 1399484
No longer depends on: 1399484
Depends on: 1399747
Depends on: 1399772
No longer depends on: 1398750
Depends on: 1399821
No longer depends on: 1398747
No longer depends on: 1399385
Depends on: 1399534
Depends on: 1399642
Depends on: 1399144
No longer depends on: 1398773
No longer depends on: 1387354
No longer depends on: 1399498
Depends on: 1400162
Depends on: 1400165
Depends on: 1379395
Depends on: 1400215
Depends on: 1400221
Depends on: 1400232
Depends on: 1400247
Depends on: 1400249
No longer depends on: 1400215
No longer depends on: 1400247
No longer depends on: 1400221
Depends on: 1400266
Depends on: 1399584
Depends on: 1401085
No longer depends on: 1400162
Depends on: 1399647
Depends on: 1401908
Depends on: 1401933
No longer depends on: 1389569
No longer depends on: 1348294
Depends on: 1401991
No longer depends on: 1401991
No longer depends on: 1389912
No longer depends on: 1402312
No longer depends on: 1399534
See Also: → 1403110
Depends on: 1402309
Depends on: 1399567
Depends on: 1403359
Depends on: 1403498
Depends on: 1403734
Depends on: 1402290
No longer depends on: 1399084
No longer depends on: 1390885
Depends on: 1404268
Depends on: 1383776
Depends on: 1404451
Depends on: 1404621
Depends on: 1404685
Depends on: 1404816
No longer depends on: 1404621
Depends on: 1405593
No longer depends on: 1405593
No longer depends on: 1404685
Depends on: 1406930
Depends on: 1407185
Priority: -- → P3
Depends on: 1368460
Depends on: 1409471
Depends on: 1408738
Depends on: 1410233
Depends on: 1411007
Depends on: 1411230
Depends on: 1409148
Depends on: 1391398
Depends on: 1398103
Depends on: 1411309
Depends on: 1415096
No longer depends on: 1415096
Depends on: 1415624
Depends on: 1417927
No longer depends on: 1403498
Depends on: 1421591
Depends on: 1402050
Depends on: 1422922
Depends on: 1398395
Depends on: 1395732
Priority: P3 → P5

The bug assignee is inactive on Bugzilla, so the assignee is being reset.

Assignee: stephen → nobody
Severity: normal → S3

Should this bug just be closed at this point?

You need to log in before you can comment on or make changes to this bug.