forked from mfittko/geo-playground
-
Notifications
You must be signed in to change notification settings - Fork 0
feat(docs): add toolbar configuration documentation and implementation examples #1
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
Open
mfittko
wants to merge
15
commits into
main
Choose a base branch
from
feature/configurable-toolbar
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n examples - Introduced documentation for the toolbar configuration feature, detailing user stories, implementation checklist, and technical details. - Added implementation examples for updating ConfigContext, ConfigModal, and Index components to support toolbar visibility and default tool selection. - Included necessary translation updates for new UI elements and settings.
…ations - Implemented toolbar visibility toggle in ConfigModal, allowing users to show or hide the toolbar. - Updated ConfigContext to manage toolbar visibility state and persist it in local storage. - Enhanced Index component to conditionally render the toolbar based on the visibility state. - Added necessary translations for toolbar visibility settings in multiple languages.
- Updated Index component to show GeometryHeader only when the toolbar is visible. - Adjusted layout calculations to account for toolbar visibility, ensuring proper spacing and layout when the toolbar is hidden. - Enhanced user experience by maintaining the header's position in the layout even when the toolbar is not displayed.
…ation checklist progress - Marked completed items in the implementation checklist for the toolbar configuration feature. - Updated documentation to reflect the current status of configuration context updates, UI updates, index component integration, translations, and key UX considerations. - Ensured clarity and completeness of the documentation for future reference and development.
- Added .npmrc settings to fix Rollup native module issues and manage optional dependencies. - Modified package.json build script to include node options for Vercel compatibility. - Introduced vercel.json for deployment configuration, specifying build and install commands.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This reverts commit 5ced157.
- Updated Rollup to version 3.29.5 in package.json and package-lock.json. - Added support for various platforms in package-lock.json, including Android, Darwin, FreeBSD, Linux, and Windows architectures. - Updated @types/estree to version 1.0.7 for improved type definitions.
…unction-playground into feature/configurable-toolbar
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces a new feature for toolbar configuration, allowing users to hide the toolbar and set a default tool. The changes span multiple files and include updates to the configuration context, UI components, and URL handling. The most important changes are detailed below:
Documentation Updates:
docs/features/toolbar-config.md: Added comprehensive documentation for the toolbar configuration feature, including user stories, implementation checklist, technical details, and key UX considerations.Implementation Examples:
docs/features/toolbar-config/implementation-example-ConfigContext.md: Provided an implementation example for updating theConfigContext.tsxto support new toolbar visibility and default tool settings.docs/features/toolbar-config/implementation-example-ConfigModal.md: Added an example for updating theConfigModal.tsxto include UI controls for toolbar visibility and default tool selection.docs/features/toolbar-config/implementation-example-Index.md: Included an example for updating theIndex.tsxcomponent to integrate toolbar visibility and default tool configuration, with conditional rendering and initialization logic.