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

Added Nothing OS themes #401

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Added Nothing OS themes #401

wants to merge 8 commits into from

Conversation

drrnb
Copy link

@drrnb drrnb commented Jan 9, 2024

The Nothing Phone default calculator doesn't have a theme matching the rest of the design so I created one for OpenCalc in both light and dark.

Community post: https://www.reddit.com/r/NothingTech/s/VV7XRvDExT

There is an APK release on my fork but thought it wouldn't hurt to see if it was something you'd be interested in incorperating in the original version so those who would like the theme can just find OpenCalc through the channels rather than a specific version.

Gradle version update as well as a few code fixes, screenshots and a tweak to the README are also present. :)

@inson1
Copy link
Contributor

inson1 commented Jan 9, 2024

@drrnb I feel like it needs more work

IMG_20240109_110008.jpg

IMG_20240109_110338.jpg

IMG_20240109_110311.jpg

@drrnb
Copy link
Author

drrnb commented Jan 9, 2024

@inson1 Ahh yes definitely going to fix those, thanks for pointing it out :)

@inson1
Copy link
Contributor

inson1 commented Jan 9, 2024

That theme(s) adds 1/8 of current size and it isnt one of the default themes

Maybe system for managing of custom themes would be better for the future if there would be more people, who would like to create its own themes - Like structure, import, show, delete

@drrnb That was just example, there is more. Would you keep it up to date, if it was added?

@inson1
Copy link
Contributor

inson1 commented Jan 9, 2024

@Darkempire78 why isnt amoled private? Is there reason? Or should I create PR?
chrome_UIsIVqdJS1

@inson1
Copy link
Contributor

inson1 commented Jan 9, 2024

@drrnb why add tostring()?

@drrnb
Copy link
Author

drrnb commented Jan 9, 2024

@inson1 It wouldn't compile without it, IDE complained 🤷‍♂️

In regards to the other comments if it's easier I could close this PR and just work on and publish a fork specifically for Nothing users with the default theme being the Nothing themes and cherry pick merges from the original repo for the rest of the app functions.

@inson1
Copy link
Contributor

inson1 commented Jan 9, 2024

@drrnb I'm not against adding it :) (also not my call xD)
How did you get the icons? Did you create them? Are they online? (that is the part of keep it up to date, if you created them)
some things should be added anyway (like changing the fond)

Shouldnt dark and light theme use same system for dark/light theme as it is set now? or it cant be used?

@Darkempire78
Copy link
Owner

Hi @drrnb, Thanks for your PR, this update will be merge, you do not need to maintain a fork. Can you polish your PR as asked by @inson1.
I will check the code and the app size as fast as possible, but I'm very busy at the very moment

@drrnb
Copy link
Author

drrnb commented Jan 9, 2024

@inson1 The icons are using the same TTF font as the rest of the text currently but those that don't currently exist I can create drawables for and tidy up the overall look of the themes although saying that, the Nothing design style is the dot font and a thin monospace style so I may just do the PI icon, infinity symbol etc.. as that and keep the standard buttons and text in the dot font.

@Darkempire78 Thanks, I will work on it within the next week and let you know here when I think it's updated to a better overall polished look. :)

@Darkempire78
Copy link
Owner

@drrnb There is a conflict with your PR, can you fix that please ? Btw is the theme finished ?

@Darkempire78
Copy link
Owner

pi and sqrt are not following the theme

@Darkempire78
Copy link
Owner

Darkempire78 commented Mar 11, 2024

Btw the Light version of the theme looks very weird. Perhpas it needs to be reworked. I found a version of Opencalc modified for Nothing phone (it is violating the licence lmao, I will report it to Google). But the colors look good. What do you think ?
unnamed-1
image

@drrnb
Copy link
Author

drrnb commented Mar 11, 2024

pi and sqrt are not following the theme

I know.
I think I need to convert the buttons to ImageButtons like the others to make them look right. Then tackle the actual display of the characters. Ran out of time last night so pushed as far as I'd got.

The issue is the Nothing font I used doesn't contain all characters so I'm having to crudely make drawable versions of the missing ones. Haha

@drrnb
Copy link
Author

drrnb commented Mar 11, 2024

RE: Light mode - Completely agree, was gonna look into colouring after I fixed the icons :)

@MillionsToOne
Copy link
Contributor

I mean, can't you like make a simple svg for PI and sqrt. Edit it in fotoshop or something and then export as svg?

@MillionsToOne
Copy link
Contributor

Or have you already fixed them

@Darkempire78
Copy link
Owner

is it still in progress?

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.

None yet

4 participants