Skip to content

Conversation

@andydotxyz
Copy link
Member

Checklist:

  • Tests included.
  • Lint and formatter run with no errors.
  • Tests all pass.

@andydotxyz andydotxyz requested a review from Jacalz July 9, 2025 17:29
@andydotxyz
Copy link
Member Author

I found that FYNE_THEME was being ignored if the xdg/portal lookup failed.
But also if it succeeds or any system change happens then the fact it was set is overwritten.

@coveralls
Copy link

Coverage Status

coverage: 62.386% (-0.006%) from 62.392%
when pulling 460ed63 on andydotxyz:fix/xdgthemefallback
into c23b700 on fyne-io:develop.

Copy link
Member

@Jacalz Jacalz left a comment

Choose a reason for hiding this comment

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

Nice :)

@andydotxyz andydotxyz merged commit 79ccd6b into fyne-io:develop Jul 11, 2025
29 of 35 checks passed
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.

3 participants