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

(Mostly) Update Aegis to Material 3 #1208

Closed
wants to merge 10 commits into from

Conversation

Wojtaz0w
Copy link

@Wojtaz0w Wojtaz0w commented Oct 7, 2023

Also update Polish translation.
I'm completely new to Android development, so there may be some mistakes. Please forgive me if there are any. Also if you want me to change something, please let me know.
Issue #1149

@Silther
Copy link

Silther commented Oct 7, 2023

Could you please split the changes to polish language from the material you support, it's a little bit confusing.

@Wojtaz0w
Copy link
Author

Wojtaz0w commented Oct 7, 2023

Tomorrow, i gtg to sleep now🙂 but yeah, sure i can

@suryaprakash251201
Copy link

suryaprakash251201 commented Oct 8, 2023 via email

Copy link

@marat2509 marat2509 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alexbakker
Copy link
Member

Thanks for the PR. This is a good start, but it looks like there's still quite a bit to do here, and we can't merge partial changes like this. We appreciate the effort, but we think switching to Material 3 is too big of a change to take in as an outside contribution. There'd be too much back and forth over small details, like what colors to use.

We should probably have replied sooner to the issue you created back in June. Sorry about that, but like our contribution guidelines states, it's usually best to wait for a proposal to be accepted before starting work on a change.

We'll pick the switch to Material 3 up ourselves soon. If you'd like, we can add you as a Co-Authored-By as a token of appreciation.

@alexbakker alexbakker closed this Nov 5, 2023
@Wojtaz0w
Copy link
Author

Wojtaz0w commented Nov 5, 2023

Cool, love wasting my time 🙃
At least I have some experience in coding on Android.
What about translation changes tho??

@alexbakker
Copy link
Member

What about translation changes tho??

We'd be happy to take those through Crowdin.

@Wojtaz0w
Copy link
Author

Wojtaz0w commented Nov 5, 2023

Also, since you didn't bother making the switch to MD3 for almost two years, I decided to take matter into my hands only for you to put me off and promise you'll do it someday (you promised already 1 and a half year ago)

@Wojtaz0w
Copy link
Author

Wojtaz0w commented Nov 5, 2023

What about translation changes tho??

We'd be happy to take those through Crowdin.

Can't you just merge my commit with translations only? Also I will never create an account on a privacy-invasive website just to translate one app.

@michaelschattgen
Copy link
Member

michaelschattgen commented Nov 5, 2023

I understand the frustration. We are working on this project in our spare time and work on things based on our own priorities. Design or Material 3 in particular never has been high on our list since it's a lot of work/effort for a little return. We rather spend our time fixing bugs or improving the user experience. We've always been transparant about the fact that we can't guarantee features or that we can't give ETA's for new updates or merges, it's ready when it's ready.

The offer to add you as a co-author to the Material 3 changes we're going to make still stands.

Can't you just merge my commit with translations only?

Unfortunately we can't make exceptions on this: https://github.com/beemdevelopment/Aegis/blob/master/CONTRIBUTING.md#translations.

@flexagoon
Copy link

flexagoon commented Nov 6, 2023

@Wojtaz0w you can checked out Authenticator Pro, which is another great open source 2FA app except it actually supports MD3, I recently switched to it from Aegis due to the lack of MD3 support in Aegis and it's been great so far (I'm not affiliated with that project in any way)

@salad612
Copy link

salad612 commented Nov 7, 2023

What about translation changes tho??

We'd be happy to take those through Crowdin.

Can't you just merge my commit with translations only? Also I will never create an account on a privacy-invasive website just to translate one app.

Surely you don't have to use your real email address to sign up for that site? You could use an aliasing service like simplelogin or even a disposable email service like 10 minute mail.

If you use a trusted VPN (or better still Tor Browser) they also won't have your IP address either and it's pretty much pseudonymous sign up if you dont use real name

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.

8 participants