scribe-org / Scribe-Android

Android app with keyboards for language learners
GNU General Public License v3.0
27 stars 14 forks source link

Add dark and light mode responsiveness #27

Closed andrewtavis closed 2 months ago

andrewtavis commented 2 years ago

Terms

Description

This issue is to implement color color scheme switching between light and dark modes.

ayushanand308 commented 1 year ago

@andrewtavis This seems a good feature. Will be working on it :).

andrewtavis commented 1 year ago

Hey @ayushanand308! Thanks for your interest in Scribe! Yes I think this would be a great feature to better understand the project 😊

I'll assign this to you, and please let me know if you have any questions!

ayushanand308 commented 1 year ago

@andrewtavis sure!

ayushanand308 commented 1 year ago

@andrewtavis Basically what I understand from the task description is that I have to implement a way to switch between dark and light themes on the keyboard right?

ayushanand308 commented 1 year ago

WhatsApp Image 2023-03-01 at 00 24 47

Thinking of implementing it like this. Is this ok?

andrewtavis commented 1 year ago

Yes, exactly @ayushanand308! Specifically it would also be good if the default was set based on the mode of the user’s phone. Not sure if you have an idea on how best to do that, and I could support with exploration if needed :) So if the user has dark mode on their phone, it would be great if the option was already selected for them 😊

ayushanand308 commented 1 year ago

@andrewtavis I would surely ask for help wherever I need it.

andrewtavis commented 1 year ago

Wonderful, @ayushanand308 :) Looking forward to the results! 😊

ayushanand308 commented 1 year ago

@andrewtavis I am not able to understand how to add functionality to this dark theme checkbox toggle like there is for "vibrate on keypress" and others. Can you help with this.:-)

https://user-images.githubusercontent.com/99035102/222565267-8374282c-f3d2-45a1-8279-1c961c63bb74.mp4

andrewtavis commented 1 year ago

Hi @ayushanand308! Just wanted to write that I have this in mind. Currently visiting family at the moment :)

ayushanand308 commented 1 year ago

@andrewtavis Yeah no issues.We can get back to this later on 😊.

ayushanand308 commented 1 year ago

https://user-images.githubusercontent.com/99035102/224350948-1bb26a91-aa85-42a5-a0e3-e1ec831171c1.mp4

The functionality of default theme being set to the user's default is done.

andrewtavis commented 1 year ago

Wonderful, @ayushanand308!! Sorry I haven’t been responsive. I’m in the midst of lots of family visits and have a wedding today, so really have been busy. Appreciate you getting to this and your dedication! I’m done with this trip Monday evening and will connect with you then.

Thanks again so much 😊

ayushanand308 commented 1 year ago

@andrewtavis Apologies for responding so late, I was caught up in my semester exams. I also wanted to ask if I can submit a GSOC proposal regarding scribe project to you for review before final proposal submission for any type of feedback.

andrewtavis commented 1 year ago

No stress, @ayushanand308! Do you want to send a PR with the work you did so far and we can figure out how to apply it to the keyboard? :)

As far GSoC, you're welcome to send along a proposal to me! You can reach me at my email in my profile here, and feel free to join us in the Matrix space we made for Scribe (there's a room for GSoC and others like Android). As GSoC is focussing on iOS, I also did a minor PR in Scribe-iOS#96f32ab to open up the Danish keyboard so you can do some work there as you'd expressed before.

ayushanand308 commented 1 year ago

@andrewtavis I have sent a PR. Can you review it and tell me if you want any changes. :-)

andrewtavis commented 1 year ago

Thanks, @ayushanand308! Will check it later today 😊 Looking forward to seeing how it's changed :)

andrewtavis commented 1 year ago

Note for this issue that #33 got dark mode working for Xiaomi Redmi 9. I was not able to reproduce on the latest Pixel, but that PR was merged for the interim progress. Thank you @ayushanand308! As I said in the PR, appreciate the code that you wrote and your willingness to help on all this. I'm sorry I couldn't focus on this as much as I could.

andrewtavis commented 2 months ago

Hey @angrezichatterbox πŸ‘‹ Do you want to write in here so I can assign for you looking further into this one?

angrezichatterbox commented 2 months ago

Ya sure .. I would work on this issue .

angrezichatterbox commented 2 months ago

The keyboard color should also change according to the current color scheme present in the schema for nightly and normal right

andrewtavis commented 2 months ago

Yes ideally that keyboard color would also change. We can work on the specifics of the colors later when we do the coloration of the special keys, so for now just make the keys black and the background a different grey :)

angrezichatterbox commented 2 months ago

I have made the pull request please do tell if there are any changes to make I would be happy to make it perfect if there are any flaws.

andrewtavis commented 2 months ago

Thanks, @angrezichatterbox! You really are making some progress here!

andrewtavis commented 2 months ago

I'll check this tonight :)

andrewtavis commented 2 months ago

Closed by #38 πŸ₯³πŸ™Œ Thanks so much for all the consistent work, @angrezichatterbox! Really amazing 😊 I'll write to you on Matrix for the next steps :)