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

Refactor colors across Scribe-iOS #216

Closed
2 tasks done
andrewtavis opened this issue Oct 3, 2022 · 7 comments
Closed
2 tasks done

Refactor colors across Scribe-iOS #216

andrewtavis opened this issue Oct 3, 2022 · 7 comments
Assignees
Labels
help wanted Extra attention is needed refactor Refactor code to improve quality

Comments

@andrewtavis
Copy link
Member

Terms

Description

As discussed in #166, the way that Scribe assigns colors is highly verbose and error prone. It would be better if the colors were defined in an asset file and accessed via an enum. This would allow us to not need to check for the traitCollection each time the color might switch based on light or dark mode.

Contribution

This issue has already been requested as part of Hacktoberfest, and I'll of course be happy to support where needed 😊

@andrewtavis andrewtavis added help wanted Extra attention is needed refactor Refactor code to improve quality labels Oct 3, 2022
@misteu
Copy link
Contributor

misteu commented Oct 3, 2022

Hey!
Thanks for creating the issue :) As discussed, I'm happy to help you out with this one.

@andrewtavis
Copy link
Member Author

Assigned! Thanks for your willingness to help out! :) I'd say this might want to wait for #210, which should be closed by tomorrow 😊

@andrewtavis
Copy link
Member Author

@misteu, wanted to let you know that the major changes that were being worked on for v2.0.0 are now all done, so the repo isn't nearly as in flux as it was 😇 Let me know if you need any help or information for this issue!

Sent the new version to Apple just a bit ago, so hopefully it'll be out in 48 hours 🚀😊

@andrewtavis
Copy link
Member Author

There's also a 1x1 conjugation/declension view that could be used for the information icon button that you implemented in #166. The issue to add a message to the user when they click that is #214 :)

misteu added a commit to misteu/Scribe-iOS that referenced this issue Oct 11, 2022
@misteu
Copy link
Contributor

misteu commented Oct 11, 2022

I created a PR for it, see #228

It looks like a lot of changes but most of it are added color asset JSONs as you will see :)

@andrewtavis
Copy link
Member Author

Thank you, @misteu! I'll take a look at all of this tomorrow :) Really looking forward to these changes and learning more about the methods you're using 😊

@andrewtavis
Copy link
Member Author

Thanks for all your work here, @misteu! Let me know if you have interest in anything else :) Big thing we're focussing on is #96 at the moment, and with that reworking the we read in data. At this point the Russian keyboard is crashing from the outset, with the current theory being that the JSONs being loaded in are much too large. I'll be looking into Core Data as suggested by another contributor 😊 You're of course welcome to work on anything that interests you, and know that your input is highly valued! Really has been great to have you 🚀

Thanks again so much 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed refactor Refactor code to improve quality
Projects
None yet
Development

No branches or pull requests

2 participants