lbryio / lbry-android

The LBRY Android app
MIT License
2.43k stars 113 forks source link

I extracted dimension values into resource files #1151. #1219

Open veritaslogix opened 3 years ago

veritaslogix commented 3 years ago

(https://github.com/lbryio/lbry-android/issues/1151)

Hi there! Because a lot of dimensions get reused, when I moved the dimensions file I used names like "large_1", "small_1", and "normal_1" for dimension values.

I tried this to cut down on repetitive values in the dimens file.

"small" values range between 0dp and 15dp "normal" values range between 16dp to 119dp "large" values are anything over 120dp (the largest is 480dp)

[PS, I submitted an application to LBRY for an android engineer position on September 9, 2021. This PR is part of my application process.]

PR Checklist

Please check all that apply to this PR using "x":

PR Type

What kind of change does this PR introduce?

Fixes

Issue Number: 1151

What is the current behavior?

What is the new behavior?

Other information

kekkyojin commented 3 years ago

For the next time, I suggest you to use a new branch for yout changes instead of commiting to your master branch

veritaslogix commented 3 years ago

Okay, thanks!