Skip to content
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

#630 Add style keys #634

Merged
merged 2 commits into from
Mar 25, 2024
Merged

#630 Add style keys #634

merged 2 commits into from
Mar 25, 2024

Conversation

DevFromDownUnder
Copy link
Contributor

Add keys to styles
Standardise style key names
Add an example of an override dictionary entry

Add keys to styles
Standardise style key names
Add an example of an override dictionary entry
@DevFromDownUnder
Copy link
Contributor Author

DevFromDownUnder commented Mar 21, 2024

Add keys to styles Standardise style key names Add an example of an override dictionary entry

Kept legacy names in case anyone is using them currently.

I just tested in the demo app, comparing against an unchanged version.

@enisn
Copy link
Owner

enisn commented Mar 21, 2024

Resolves #630

@enisn
Copy link
Owner

enisn commented Mar 22, 2024

Some tests are failed
image

@DevFromDownUnder
Copy link
Contributor Author

Some tests are failed image

See this comment

#635 (comment)

@DevFromDownUnder
Copy link
Contributor Author

Reset and branched that extra commit, should be good now

@DevFromDownUnder
Copy link
Contributor Author

There must be a bug in VS, that last commit was pushed to a branch, it should not have been picked up here

@DevFromDownUnder
Copy link
Contributor Author

DevFromDownUnder commented Mar 25, 2024

There must be a bug in VS, that last commit was pushed to a branch, it should not have been picked up here

Fixed, can you merge/close this one off if you have completed it

@enisn enisn merged commit 110ae6c into enisn:develop Mar 25, 2024
3 checks passed
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.

2 participants