Skip to content

Draft operators notification icon - #314

Merged
stewit merged 3 commits into
developfrom
draft-operators-notification-icon
Nov 6, 2026
Merged

stewit merged 3 commits into
developfrom
draft-operators-notification-icon

Conversation

@stewit

@stewit stewit commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

No description provided.

stewit added 3 commits November 6, 2026 17:28
Reloading while the fonts of the first page load are still downloading
makes firefox abort the downloads and log "downloadable font: download
failed" console errors, which fail the test. Exactly the specs that
reload the page failed this way in firefox. They now reload via
HetidaDesigner.reloadPage(), which waits for document.fonts.ready first.

Assisted-by: Claude Code:claude-opus-5-5
All notifications are rendered by the new NotificationMessageComponent
instead of the simple snack bar of Angular Material. Besides plain
text, a message can be a list of text and icon parts, e.g. to show the
toolbar button the message refers to.

- Clicking a message closes it, unless the click selected text of it.
- A message is shown 5 seconds plus one second per 20 characters of
  its text as shown (typical reading speed of about 1200 characters per
  minute), at most 20 seconds.
- The background is slightly transparent, so that the toolbar buttons
  underneath remain recognizable. 80 % opacity keeps the contrast of
  the white text at 4.5:1 or more.

Assisted-by: Claude Code:claude-opus-5-5
Publishing a workflow with DRAFT operators is refused with a warning
which referred to "the respective toolbar button". It now shows the
icon and the name of the "Upgrade workflow operators" button.

The new e2e test checks that the warning shows the same icon as the
toolbar button and that clicking the warning closes it.

Assisted-by: Claude Code:claude-opus-5-5
@stewit
stewit merged commit 93dfe46 into develop Nov 6, 2026
3 checks passed
@stewit
stewit deleted the draft-operators-notification-icon branch November 6, 2026 17:02
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