-
Notifications
You must be signed in to change notification settings - Fork 68
Fix/organizer-dashboard. Allow organizer to preview the Event page customization through a modal #770
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/organizer-dashboard. Allow organizer to preview the Event page customization through a modal #770
Conversation
…ewModal component to show preview of customized event page
…ge customizations
…ug/index page, subject to modification for preview only purposes
…EventPageCustomizationPreviewModal inside _organize/slug/edit/default component
…inside _organize/slug/edit/default component as the EventPagePreview component will be directly rendered in the modal
…eters to read and set Preview mode
… deatil preview modal
…ton based on state
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @davidamebley ,
Well done! 🎉
Your solution works well, and it is scheduled for merge on February 28th
Highlights
- Clean solution to the problem
Optional suggestions
- The preview doesn't show new colors selected by the user, unless they save their color choices. It would be a better user experience if the colors selected could be previewed without having to save
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Update - Bringing #770 with merge conflicts solved
Add a modal that allows the organizer to preview the Event page customization without leaving the current page.