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

Compact more info header #15448

Merged
merged 10 commits into from
Feb 20, 2023
Merged

Compact more info header #15448

merged 10 commits into from
Feb 20, 2023

Conversation

piitaya
Copy link
Member

@piitaya piitaya commented Feb 14, 2023

Proposed change

Tabs has been replaced by icons and overflow menu :

  • more compact
  • info view is now the main view and each other view have a back button
  • No more issue with long words in tabs

Demo

CleanShot.2023-02-16.at.14.41.15.mp4

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@piitaya piitaya force-pushed the more-info-navigation branch 4 times, most recently from e34d266 to 4484b86 Compare February 16, 2023 13:32
@piitaya piitaya marked this pull request as ready for review February 16, 2023 13:43
@piitaya piitaya changed the title New more-info navigation Compact more info header Feb 16, 2023
@piitaya piitaya force-pushed the more-info-navigation branch from 9975044 to 24287f9 Compare February 16, 2023 16:41
@piitaya piitaya added this to the 2023.3 milestone Feb 20, 2023
package.json Outdated Show resolved Hide resolved
bramkragten
bramkragten previously approved these changes Feb 20, 2023
@bramkragten bramkragten enabled auto-merge (squash) February 20, 2023 13:56
@bramkragten bramkragten merged commit 8754947 into dev Feb 20, 2023
@bramkragten bramkragten deleted the more-info-navigation branch February 20, 2023 13:58
@antonio1475
Copy link

@piitaya and all thank you for your work on this topic - amazing new dialog!!

Will you please think/work on the usability when it comes to closing the dialog on mobile (iOS)?
On desktop it's easy - just click outside the dialog. However, on mobile one has to reach to top-left for the close button.

Something like a close button at the bottom or making it more a popup than a full-screen dialog could work?

Thank you again and apologies for the intrusion.

@github-actions github-actions bot locked and limited conversation to collaborators Feb 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants