Skip to content
This repository has been archived by the owner on Feb 7, 2019. It is now read-only.

DR: System-specific styling needs for doorhanger #662

Open
changecourse opened this issue Mar 15, 2018 · 5 comments
Open

DR: System-specific styling needs for doorhanger #662

changecourse opened this issue Mar 15, 2018 · 5 comments

Comments

@changecourse
Copy link
Contributor

Alignment of our doorhanger to corresponding system defaults within the browser.

On macOS and Linux doorhangers have rounded corners. (2px corner radius)

On Windows doorhangers have sharp corners.

@jimporter
Copy link
Contributor

jimporter commented Mar 15, 2018

As far as I can tell, Linux actually uses square corners everywhere. It's just Mac that's the weird one. Is this wrong?

Is there any implementation work to do here, or is this just a design task? The last I checked, we don't have any control over this, since it's something Firefox does.

@changecourse
Copy link
Contributor Author

I was pulling from the spec... but if Linux doesn't do rounded corners for default firefox doorhangers, then I'll address with the Photon team...

https://design.firefox.com/photon/components/doorhangers.html

@jimporter I put this in as an implementation task, but I guess that's an open question; is our app supposed to identify the current system and inherit any specific styles like this?

@jimporter
Copy link
Contributor

jimporter commented Mar 15, 2018

@changecourse Yeah, let's file an issue for the Linux thing, since I don't see rounded corners on Linux and I don't think I ever have.

As for the doorhangers themselves, I don't think we have control over that. From the images people on macOS post, it seems like the corners are (used to be?) rounded. For example, this one from 2 weeks ago looks good. However, a more-recent one looks bad (from 3 days ago).

Maybe there's a Firefox bug? As far as I know we haven't changed anything that would affect this between now and then. We could always try an older build just to be sure we didn't break this tough.

All that said, there may be some merit in doing platform-specific styles on the account details dropdown in the management UI. That's supposed to look doorhanger-ish, right?

@changecourse
Copy link
Contributor Author

Hmmmmm.... I did a little more digging, and installed SnoozeTabs via TestPilot... They now have squared off corners too on MacOS, and that was not the case previously (as far as I can recall)

I'll try and see if anyone else is experiencing this or if a Fx bug exists.

All that said, there may be some merit in doing platform-specific styles on the account details dropdown in the management UI. That's supposed to look doorhanger-ish, right?

Do you mean for the full-tab management UI? Are there areas in this view that feel too aligned to MacOS, is that what you're saying?

@jimporter
Copy link
Contributor

Do you mean for the full-tab management UI? Are there areas in this view that feel too aligned to MacOS, is that what you're saying?

Yes. I don't have a strong opinion on if it's "right" or not, but if we wanted that little dropdown to look like a "native" doorhanger, then this issue would probably be a good place to work on that.

@changecourse changecourse added backlog We have looked at and understand the issue. Prioritized with alignment to product initiatives. and removed to do labels Mar 21, 2018
@devinreams devinreams added feature needs-design and removed backlog We have looked at and understand the issue. Prioritized with alignment to product initiatives. labels Mar 26, 2018
@devinreams devinreams changed the title DR: System-specific styling needs for our DH DR: System-specific styling needs for doorhanger Mar 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants