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

feat: add ShareUrlButton functionality #823

Open
wants to merge 14 commits into
base: v2
Choose a base branch
from

Conversation

mguellsegarra
Copy link
Collaborator

https://github.com/gisce/webclient/issues/1612

  • Introduced ShareUrlButton and ActionBarSeparator components across DashboardActionBar, FormActionBar, GraphActionBar, and TreeActionBar for improved UI consistency.
  • Updated locale files (ca_ES, en_US, es_ES) to include new translations for sharing functionality.
  • Enhanced type definitions to support action_id and action_type in view data structures.

gisce/webclient#1612
- Introduced ShareUrlButton and ActionBarSeparator components across DashboardActionBar, FormActionBar, GraphActionBar, and TreeActionBar for improved UI consistency.
- Updated locale files (ca_ES, en_US, es_ES) to include new translations for sharing functionality.
- Enhanced type definitions to support action_id and action_type in view data structures.
gisce/webclient#1612
- Refactored the copyToClipboard function to use a temporary textarea for better compatibility across browsers and avoid SSL restrictions
- Updated the ShareUrlButton component to include new icons and improved styling.
- Added new translations for "Copy to clipboard" in ca_ES, en_US, and es_ES locale files.
- Adjusted the popover content padding for a cleaner UI.
gisce/webclient#1612
- Updated GraphActionBar to accept a new optional `domain` prop for improved functionality.
- Modified ShareUrlButton to utilize the `domain` prop, allowing for better sharing context.
- Refactored TreeActionBar to compute `finalDomain` using `useMemo`, optimizing performance.
- Adjusted share URL creation in shareUrlHelper to include `domain` in the URL parameters if provided.
- Updated GraphActionView to pass the merged `domain` to GraphActionBar.
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.

1 participant