Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Publish view alternate layout #948

Closed
wants to merge 3 commits into from

Conversation

ptbrowne
Copy link
Collaborator

@ptbrowne ptbrowne commented Jan 27, 2023

I wonder if it could not be helpful to have the publish actions
as part of a side pane instead of below the chart. It would
make share/publish actions more prominent, especially on smaller
screens where right now, the publish actions are below the fold.

Normal layout Alternative Layout
image image

How to test

Go to https://visualization-tool-nnrpkidag-ixt1.vercel.app/en/v/sO8qHplebSER?dataSource=Prod

@vercel
Copy link

vercel bot commented Jan 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
visualization-tool ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 27, 2023 at 1:59PM (UTC)

@ptbrowne ptbrowne changed the title feat/publish view alternate view Publish view alternate layout Jan 27, 2023
@ptbrowne ptbrowne changed the base branch from main to feat/embed-options January 27, 2023 13:57
@ptbrowne ptbrowne mentioned this pull request Jan 27, 2023
@ptbrowne
Copy link
Collaborator Author

Closing in favor of the top action bar implemented in the #947 PR.

@ptbrowne ptbrowne closed this Jan 30, 2023
@AnninaWalker
Copy link
Contributor

Standard: Provides Metadata and Downloadlinks for the dataset
Minimal: Chart only with link to full information on visualize.admin.ch

Standard: Stellt Metadata und Downloadlink für das Datenset zur Verfügung
Minimal: Nur Visualisierung mit Link zu mehr Informationen auf visualize.admin.ch

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

Successfully merging this pull request may close these issues.

None yet

2 participants