Skip to content

Conversation

@sfc-gh-dmatthews
Copy link
Contributor

@sfc-gh-dmatthews sfc-gh-dmatthews commented Jul 6, 2023

The In-app menu was changed from a hamburger to a kebab. This commit updated the images on the App Menu page of the docs.

📚 Context

This is a fast-follow to update images for the new app menu documentation.

🧠 Description of Changes

Images showing the menu icon are replaced to show the new icon (/library/advanced-features/app-menu)

Revised

https://deploy-preview-730--streamlit-docs.netlify.app/library/advanced-features/app-menu

💥 Impact

Size:

  • Small

🌐 References

Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

In-app menu was change from a hamburger to a kebab. This commit updated the images on the App Menu page of the docs.
@sfc-gh-dmatthews sfc-gh-dmatthews marked this pull request as ready for review July 6, 2023 23:55
@sfc-gh-dmatthews
Copy link
Contributor Author

Since this is just an image update, it can easily wait for the matching feature to get published without creating a conflict.

Copy link
Contributor

@sfc-gh-jgarcia sfc-gh-jgarcia left a comment

Choose a reason for hiding this comment

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

LGTM 👍 FYI, the open-source changes were merged yesterday, so they should go live on the next release

@sfc-gh-dmatthews
Copy link
Contributor Author

I've searched through the site for any other references to "hamburger" or "☰" and found about a dozen instances. One question: do we want to update where the changelog has mentioned the "hamburger" menu? I can see an argument for not touching anything in the changelog, but also for making sure those historical changes are recorded in icon-agnostic language except when explicitly saying the icon was changed. (e.g. Changing "cleaned up the hamburger menu" to "cleaned up the app menu") I have the edits on my local git, ready to push when I have confirmation on this point.

@snehankekre
Copy link
Contributor

One question: do we want to update where the changelog has mentioned the "hamburger" menu?

The changelog should be historically accurate. Let's avoid rewriting it. In the next 1.25.0 changelog, we can mention the name change.

This commit replaces any reference to "hamburger" or "☰" in the docs. Older references to a hamburger menu in the Community Cloud admin panel were also removed as that icon was previously changed to a vertical ellipse also.
Update wording to indicate "click an icon" to "open a menu" (as opposed to "click a menu").
@sfc-gh-dmatthews sfc-gh-dmatthews added the release Pull requests related to the next release label Jul 14, 2023
@snehankekre snehankekre merged commit 818892c into main Jul 21, 2023
@snehankekre snehankekre deleted the feature/update-hamburger-menu branch July 21, 2023 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release Pull requests related to the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants