Skip to content

fix(activity): use window text color for scroll button - #10746

Open
Rello wants to merge 1 commit into
masterfrom
codex/fix-10740-scroll-to-top-contrast
Open

fix(activity): use window text color for scroll button#10746
Rello wants to merge 1 commit into
masterfrom
codex/fix-10740-scroll-to-top-contrast

Conversation

@Rello

@Rello Rello commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Use the client鈥檚 dark-mode-aware foreground

fixes #10740

Assisted-by: Codex:GPT-5

Signed-off-by: Rello <github@scherello.de>
Assisted-by: Codex:GPT-5
@Rello

Rello commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator Author

/backport to stable-34.0

@sonarqubecloud

sonarqubecloud Bot commented Sep 4, 2026

Copy link
Copy Markdown

Accessible.onPressAction: scrollToTopButton.clicked()

icon.source: "image://svgimage-custom-color/chevron-double-up.svg/" + palette.buttonText
icon.source: "image://svgimage-custom-color/chevron-double-up.svg/" + palette.windowText

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

buttontext should work, if it doesn't there is a broader issue with the palette here

@Rello
Rello marked this pull request as draft September 4, 2026 14:59
@mgallien

mgallien commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator
image dark mode on windows

@mgallien

mgallien commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

light mode
image

@Rello
Rello marked this pull request as ready for review September 7, 2026 12:26
@Rello

Rello commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator Author

/backport to stable-34.0

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Artifact containing the AppImage: nextcloud-appimage-pr-10746.zip

Digest: sha256:0e8a39a999b04c3be62c5e488ca9782f2f12d99f22daa253c8235d25782b48a9

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

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.

Dark theme: "Back to Top" button not obvious and misleading

3 participants