Skip to content

Improve flow modal buttons (better error feedback)#8625

Merged
nielsm5 merged 3 commits intomasterfrom
fix/mermaid-flow-modal-buttons-firefox-attempt
Mar 20, 2025
Merged

Improve flow modal buttons (better error feedback)#8625
nielsm5 merged 3 commits intomasterfrom
fix/mermaid-flow-modal-buttons-firefox-attempt

Conversation

@Matthbo
Copy link
Copy Markdown
Member

@Matthbo Matthbo commented Mar 18, 2025

Firefox can now open flows in a new tab, downloading or copying too big of a flow image still doesn't work. I give up on trying to put more energy into it.

Firefox can now open flows in a new tab, downloading or copying too big of a flow image still doesn't work. I give up on trying to put more energy into it.
@Matthbo Matthbo requested a review from philipsens March 18, 2025 17:30
@Matthbo Matthbo self-assigned this Mar 18, 2025
@Matthbo Matthbo changed the title Attempts to improve flow modal buttons Improve flow modal buttons (better error feedback) Mar 18, 2025
@Matthbo Matthbo linked an issue Mar 18, 2025 that may be closed by this pull request
@sonarqubecloud
Copy link
Copy Markdown

@philipsens
Copy link
Copy Markdown
Member

Could this benefit from an e2e test?

@nielsm5 nielsm5 merged commit dd1a83d into master Mar 20, 2025
26 checks passed
@nielsm5 nielsm5 deleted the fix/mermaid-flow-modal-buttons-firefox-attempt branch March 20, 2025 12:35
@nielsm5
Copy link
Copy Markdown
Member

nielsm5 commented Mar 20, 2025

Could this benefit from an e2e test?

Probably yes, but I think we've spent enough time on this for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New tab button of flow diagram doesn't work in Firefox

3 participants