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

Feature ::: Android ::: Edge-to-Edge in all Android versions #40

Merged
merged 5 commits into from
Nov 27, 2024

Conversation

OS-pedrogustavobilro
Copy link
Collaborator

@OS-pedrogustavobilro OS-pedrogustavobilro commented Nov 26, 2024

Description

The barcode library has edge-to-edge working on Android 15 (#37).

This PR adds the same support for other Android versions, to have consistency and also improved UI.

Context

Reference: https://outsystemsrd.atlassian.net/wiki/spaces/RDME/pages/4426137659/iOS+18+Android+15+Assessment

Type of changes

  • Fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Refactor (cosmetic changes)
  • Breaking change (change that would cause existing functionality to not work as expected)

Platforms affected

  • Android
  • iOS
  • JavaScript

Tests

Tested Barcode Sample app on different Android versions and screen sizes, to confirm that edge-to-edge is working.

Screenshots (if appropriate)

Type Before After
Phone Android 12 (portrait) before_android12_portrait after_android12_portrait
Tablet Android 10 (landscape) before_tablet_android10_landscape after_tablet_android10_lanscape

Checklist

  • Pull request title follows the format RNMT-XXXX <title>
  • Code follows code style of this project
  • CHANGELOG.md file is correctly updated
  • Changes require an update to the documentation
    • Documentation has been updated accordingly

@OS-pedrogustavobilro OS-pedrogustavobilro self-assigned this Nov 26, 2024
@OS-pedrogustavobilro OS-pedrogustavobilro requested a review from a team as a code owner November 26, 2024 13:56
@OS-pedrogustavobilro OS-pedrogustavobilro marked this pull request as draft November 26, 2024 13:56
@OS-pedrogustavobilro OS-pedrogustavobilro changed the base branch from main to development November 26, 2024 13:56
@@ -7,5 +7,5 @@
<modelVersion>4.0.0</modelVersion>
<groupId>com.github.outsystems</groupId>
<artifactId>osbarcode-android</artifactId>
<version>1.1.5</version>
<version>1.2.0-dev1</version>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if the version should be 1.1.6 instead of 1.2.0 since this is not exactly a feature we're adding.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Understood, it is a small change. I was going off of inappbrowser plugin that had a minor version update over a similar change.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh ok. It's not super important either way. It technically isn't a fix but rather a "feature" so we can actually raise the minor version.

@OS-pedrogustavobilro OS-pedrogustavobilro merged commit c9a7eb4 into development Nov 27, 2024
7 checks passed
@OS-pedrogustavobilro OS-pedrogustavobilro deleted the feature/edge-to-edge branch November 27, 2024 10:48
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