Skip to content
This repository was archived by the owner on Dec 20, 2024. It is now read-only.

Popup "designer" or "styler" #45

Closed
asangma opened this issue Mar 7, 2019 · 6 comments
Closed

Popup "designer" or "styler" #45

asangma opened this issue Mar 7, 2019 · 6 comments

Comments

@asangma
Copy link

asangma commented Mar 7, 2019

We currently have the "configure popup" icon.
image

This is a good one and has a technical feel befitting the current and legacy ways of exposing this to the user.

Our future map authoring app will have an experience that is more like "popup design", "popup style", or "popup layout".

Would we be able to make a similar icon to "configure" that is less technical and more about "styling" or "designing" a popup?

Thanks!

cc @johnmgriffith @KatelynSeitz

@SkyeSeitz
Copy link
Contributor

Cool. So there is still a need for the existing configure-popup icon as is, but in addition, there is a need for a style-popup/ design-popup as well?

We can probably leverage the styling/design pallet to some degree (rough sketch below). Sounds like we can get a request open for a new icon and go from there.
image

@asangma
Copy link
Author

asangma commented Mar 8, 2019

I think the configure may eventually be replaced by "design", but for now we would want to keep both.

I think we may want to update the popup portion of the icon too and simplify it. Partly because the popups on web apps tend to stack content vertically (and not so much like a mini webpage).

Here's an example:
popup eg

re: palette / brush
It would be good to explore something closer to "design" and further from "art". Maybe that's like a right-angle square, ruler, or that sort of concept.

:)

@SkyeSeitz
Copy link
Contributor

Gotcha. Let's address all three popup icons in the request then:
popup.svg
configure-popup.svg
design-popup.svg

A simplification that is still future UI proof would be good. 👍🏽

@asangma
Copy link
Author

asangma commented Mar 8, 2019

Sweet!

@asangma
Copy link
Author

asangma commented Mar 18, 2019

Let this fall off. Sorry.
Request submitted.

@SkyeSeitz
Copy link
Contributor

resolved in v1.6.6

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants