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

#2342 : Multimeter incomplete problem resolve and removed Hardcoded value #2394

Closed
wants to merge 4 commits into from

Conversation

Aayush1357
Copy link

@Aayush1357 Aayush1357 commented Mar 22, 2024

#2342 Multimeter screen layout incomplete

Changes

app/src/main/res/layout-hdpi/activity_multimeter.xml
app/src/main/res/layout-sw600dp/activity_multimeter.xml
app/src/main/res/layout-xhdpi/activity_multimeter.xml
app/src/main/res/layout/activity_multimeter.xml
app/src/main/res/values/dimens.xml
app/src/main/res/values/strings.xml

Screenshots / Recordings

https://github.com/fossasia/pslab-android/assets/111655141/83612119-a664-4c48-abe1-60124730ccdf
Screenshot_20240323_013925

Checklist

  • No hard coding: I have used resources from strings.xml, dimens.xml and colors.xml without hard coding any value.
  • Code reformatting: I have reformatted code and fixed indentation in every file included in this pull request.
  • No extra space: My code does not contain any extra lines or extra spaces than the ones that are necessary.

@Aayush1357 Aayush1357 changed the title #2342 : Multimeter incomplete problem resolve #2342 : Multimeter incomplete problem resolve and removed Hardcoded value Mar 25, 2024
Copy link
Collaborator

@CloudyPadmal CloudyPadmal left a comment

Choose a reason for hiding this comment

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

There are still hardcoded dimensions in many place. Please go through line by line and fix. I have only marked a few. Also use sensible variable names when naming those dimensions and avoid things like margin_1 etc.

Comment on lines 189 to 190
android:layout_marginStart="5dp"
android:layout_marginTop="5dp"
Copy link
Collaborator

Choose a reason for hiding this comment

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

hardcoded

android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_gravity="center"
android:layout_marginTop="4dp"
Copy link
Collaborator

Choose a reason for hiding this comment

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

hardcoded

@AsCress
Copy link
Contributor

AsCress commented May 16, 2024

Resolved by #2407

@AsCress AsCress closed this May 16, 2024
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

3 participants