fix(ui): move blocked images notice out of the message body - #13463
Open
CreatorOfBiontium wants to merge 2 commits into
Open
fix(ui): move blocked images notice out of the message body#13463CreatorOfBiontium wants to merge 2 commits into
CreatorOfBiontium wants to merge 2 commits into
Conversation
The notice was inside the message's white body, where --color-text-maxcontrast is barely readable. Render it as its own section above the body, like the phishing warning, so it reads on the app background. Extract it into BlockedContentWarning and drop the :deep button overrides it needed, which also leaked onto the translate and read receipt buttons. AI-assisted: Claude Code (Claude Opus 5) Signed-off-by: Tobiáš Vašťák <tobias.vastak@gmail.com>
CreatorOfBiontium
requested review from
ChristophWurst,
GretaD and
kesselb
as code owners
August 11, 2026 15:45
Member
|
Thanks for the contribution |
Contributor
|
Thanks a lot! Can you take a look at the conflicts? |
Contributor
|
@GVodyanov do you think we should port that style to the translate hint? |
Contributor
Author
Ofc :) but I'm not home now, I'll do it on saturday |
The notice moved into its own component while main renamed the Actions components, added the iframe sandbox and swapped the translation check for browser-side language detection. Keep the extracted warning and take main's version of everything it sits next to. Assisted-by: ClaudeCode:claude-opus-5 Signed-off-by: Tobiáš Vašťák <tobias.vastak@gmail.com>
Contributor
Author
|
@kesselb conflicts solved |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #12075
Fixed the barely readable "The images have been blocked to protect your privacy." section.
The notice was inside the message's white body, where --color-text-maxcontrast was barely readable. Now we render it as its own section above the body, like the phishing warning, so it reads on the app background.
🤖 AI (if applicable)