mirror of
https://github.com/webstudio-is/webstudio.git
synced 2026-11-06 03:58:34 +09:00
e806bcbf5c6e06edb2bb20dec24c836e95cb63cb
## Description Close the dialog when a navigable link within it is clicked Fixes the issue when we have anchor links inside Sheet ## Steps for reproduction ### Project Example: https://href.staging.webstudio.is/builder/95821bd9-040a-4294-aa7f-edf26d7f1ebe ### Published: https://link-inside-sheet-saas-sk7pk.wstd.work/ Open sheet Click links, links should work ## Code Review - [ ] hi @kof, I need you to do - conceptual review (architecture, feature-correctness) - detailed review (read every line) - test it on preview ## Before requesting a review - [ ] made a self-review - [ ] added inline comments where things may be not obvious (the "why", not "what") ## Before merging - [ ] tested locally and on preview environment (preview dev login: 5de6) - [ ] updated [test cases](https://github.com/webstudio-is/webstudio/blob/main/apps/builder/docs/test-cases.md) document - [ ] added tests - [ ] if any new env variables are added, added them to `.env.example` and the `builder/env-check.js` if mandatory
Learning Resources
- Blog
- Documentation
- Brand and Product Design
- Contributing Guide for Devs
- Github Discussions
- Wishlist
- Builder Issues Tracker
- Roadmap
Social Media
Thanks
Thanks to Lost Pixel for providing the visual testing platform that helps us review UI changes and catch visual regressions.
Languages
TypeScript
98.9%
PLpgSQL
0.5%
CSS
0.3%
Shell
0.2%
JavaScript
0.1%
