Skip to content

Fixed app crash #728

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 4, 2020
Merged

Fixed app crash #728

merged 2 commits into from
May 4, 2020

Conversation

tsvietOK
Copy link
Contributor

Var TintLuminosityOpacity removed as it has the same value(0.9) for light and dark themes.
Renamed class
Fixes #720

Var TintLuminosityOpacity removed as it has the same value(0.9) for light and dark themes.
Renamed class
@ghost ghost added the needs - code review label Apr 30, 2020
@yaira2
Copy link
Member

yaira2 commented May 1, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yaira2 yaira2 requested a review from lukeblevins May 4, 2020 16:02
@lukeblevins
Copy link
Contributor

Thanks for this PR. Could you please explain how this one fixes the crash?

@tsvietOK
Copy link
Contributor Author

tsvietOK commented May 4, 2020

@duke7553 Read my comment here

@tsvietOK
Copy link
Contributor Author

tsvietOK commented May 4, 2020

@duke7553 May be you can realize possible reasons why setting a value to var causes app crash in release mode

@yaira2 yaira2 merged commit 98799df into files-community:develop May 4, 2020
@yaira2 yaira2 added ready to merge Pull requests that are approved and ready to merge and removed needs - code review labels May 4, 2020
@tsvietOK tsvietOK deleted the theme-fix branch May 4, 2020 16:13
BuraChuhadar pushed a commit to BuraChuhadar/files-uwp that referenced this pull request May 18, 2020
Var TintLuminosityOpacity removed as it has the same value(0.9) for light and dark themes.
Renamed class
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Pull requests that are approved and ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants