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 SVG resource file for Icon #702

Closed
wants to merge 1 commit into from

Conversation

Akash-Ramjyothi
Copy link
Contributor

Fixes: #701

Description:

Replaced the existing PNG (Portable Network Graphics) resource file with an existing SVG (Scalable Vector Graphics) resource file without altering the Texture to improve scalability across various screens.

Link for currently used PNG:
https://github.com/enviroCar/enviroCar-app/blob/master/org.envirocar.app/res/drawable-hdpi/ic_drive_eta_black_24dp.png

Link for SVG:
https://github.com/enviroCar/enviroCar-app/blob/master/org.envirocar.app/res/drawable/ic_directions_car_black_24dp.xml

@Akash-Ramjyothi Akash-Ramjyothi changed the base branch from develop to gsoc2021 June 16, 2021 18:15
@Akash-Ramjyothi
Copy link
Contributor Author

@SebaDro sir please review it.

@Akash-Ramjyothi
Copy link
Contributor Author

@SebaDro @arvindnegi1 @bpross-52n can you please review it before it gets superseded?

Copy link
Member

@asaikarthikeya asaikarthikeya left a comment

Choose a reason for hiding this comment

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

LGTM, Valid point. I have tested this in a Tablet. There are few other areas were ic_drive_eta_black_24dp is still used. You can consider to change them also. Please rebase onto develop. Thankyou for the PR @Akash-Ramjyothi ;)

@Akash-Ramjyothi Akash-Ramjyothi changed the base branch from gsoc2021 to develop August 5, 2021 09:28
@Akash-Ramjyothi
Copy link
Contributor Author

@asaikarthikeya I have successfully rebased it onto develop. Thank you for approving. Can you please Merge this PR?

@SebaDro
Copy link
Contributor

SebaDro commented Jul 25, 2023

Will be closed due to the same reason as #898

@SebaDro SebaDro closed this Jul 25, 2023
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.

3 participants