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

Fix there's no publish button on mobile query page #4760

Merged
merged 3 commits into from
Mar 30, 2020

Conversation

gabrieldutra
Copy link
Member

What type of PR is this? (check all applicable)

  • Bug Fix

Description

Reported by @susodapop, the Publish Button was removed during the Query Page migration since there was too many buttons and it was breaking the width layout. But since #4663 the buttons wrap in a new row, so we can fit the Publish button back.

If you think there are too many buttons on mobile version it's also possible to move the Publish button to the dropdown.

Related Tickets & Documents

--

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Query View

Before After
Screen Shot 2020-03-29 at 12 57 39 Screen Shot 2020-03-29 at 12 55 23

Query Source

Before After
Screen Shot 2020-03-29 at 12 57 20 Screen Shot 2020-03-29 at 12 56 34

@gabrieldutra
Copy link
Member Author

gabrieldutra commented Mar 29, 2020

Update: I decided to proceed with "If you think there are too many buttons on mobile version it's also possible to move the Publish button to the dropdown." as it looked to me the Query View buttons could overflow on smaller mobile screens (the view page has the "Refresh" button too). So the result is now:

query-publish-mobile

Copy link
Collaborator

@kravets-levko kravets-levko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

client/app/pages/queries/components/QueryPageHeader.jsx Outdated Show resolved Hide resolved
@gabrieldutra gabrieldutra merged commit e3745f8 into master Mar 30, 2020
@gabrieldutra gabrieldutra deleted the query-publish-mobile branch March 30, 2020 21:16
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.

2 participants