Skip to content
This repository was archived by the owner on Feb 11, 2025. It is now read-only.

bug/fix resources#1

Merged
javiergm1983 merged 8 commits into
developfrom
bug/fix-resources
Oct 20, 2020
Merged

bug/fix resources#1
javiergm1983 merged 8 commits into
developfrom
bug/fix-resources

Conversation

@Gnzlt

@Gnzlt Gnzlt commented Oct 13, 2020

Copy link
Copy Markdown

What does this implement/fix? Explain your changes.

  • Added new scan icon
  • Compressed existing resources
  • Replaced hardcoded Strings with placeholders
  • Added context to new Strings to be translated.

Any relevant logs, error output, etc?

...

Any other comments?

...

Where has this been tested?

  • Devices/Simulators:
    Google Pixel 4
    Google Pixel XL

  • OS Version:
    Android 11
    Android 10

@Gnzlt Gnzlt requested a review from javiergm1983 October 13, 2020 15:24
@Gnzlt Gnzlt self-assigned this Oct 13, 2020
Comment thread documentscanner/build.gradle
@Gnzlt Gnzlt requested a review from javiergm1983 October 15, 2020 13:27
Comment thread documentscanner/src/main/res/values/strings.xml
@Gnzlt Gnzlt requested a review from javiergm1983 October 19, 2020 13:12

@javiergm1983 javiergm1983 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Some strings improvements needed. Please take a look to my comments.

In addition there is a problem with other string. As you can see in the following screenshot it says "Upload" where it should say "Add" (see the design).

Apart from that everything seems fine.

Screenshot_20201019-175719

Comment thread documentscanner/src/main/res/values/strings_document_scanner.xml Outdated
Comment thread documentscanner/src/main/res/values/strings_document_scanner.xml Outdated
Comment thread documentscanner/src/main/res/values/strings_document_scanner.xml
@Gnzlt Gnzlt requested a review from javiergm1983 October 20, 2020 08:23

@javiergm1983 javiergm1983 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I have seen some other problems with the strings. Sorry for not seeing and reporting them yesterday along with the other issues.

Comment thread documentscanner/src/main/res/values/strings_document_scanner.xml Outdated
Comment thread documentscanner/src/main/res/values/strings_document_scanner.xml Outdated
Comment thread documentscanner/src/main/res/values/strings_document_scanner.xml Outdated
@Gnzlt Gnzlt requested a review from javiergm1983 October 20, 2020 09:55
@Gnzlt Gnzlt added the bug Something isn't working label Oct 20, 2020
@javiergm1983 javiergm1983 requested review from ash-mega and removed request for ash-mega October 20, 2020 11:53
@javiergm1983 javiergm1983 merged commit 303814e into develop Oct 20, 2020
@javiergm1983 javiergm1983 deleted the bug/fix-resources branch October 20, 2020 12:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants