Skip to content

Place buttons one below the other in the add repeat dialog#6977

Merged
seadowg merged 1 commit intogetodk:masterfrom
grzesiek2010:COLLECT-6976
Dec 1, 2025
Merged

Place buttons one below the other in the add repeat dialog#6977
seadowg merged 1 commit intogetodk:masterfrom
grzesiek2010:COLLECT-6976

Conversation

@grzesiek2010
Copy link
Member

@grzesiek2010 grzesiek2010 commented Nov 26, 2025

Closes #6976

Why is this the best possible solution? Were any other approaches considered?

As discussed in the issue, for some translations or on smaller screens, displaying the buttons in a row, even when expanded, might not be sufficient. Changing the layout to vertical is a safer solution.

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

It should only change the button layout in the dialog from horizontal to vertical.

Do we need any specific form for testing your changes? If so, please attach one.

Any form with repeats.

Does this change require updates to documentation? If so, please file an issue here and include the link below.

No.

Before submitting this PR, please make sure you have:

  • added or modified tests for any new or changed behavior
  • run ./gradlew connectedAndroidTest (or ./gradlew testLab) and confirmed all checks still pass
  • added a comment above any new strings describing it for translators
  • added any new strings with date formatting to DateFormatsTest
  • verified that any code or assets from external sources are properly credited in comments and/or in the about file.
  • verified that any new UI elements use theme colors. UI Components Style guidelines

@grzesiek2010 grzesiek2010 requested a review from seadowg November 28, 2025 13:27
@seadowg seadowg merged commit 3df2b55 into getodk:master Dec 1, 2025
7 checks passed
@dbemke
Copy link

dbemke commented Dec 3, 2025

Tested with Success!

Verified on devices with Androids: 8.1, 10 and 16

Verified cases:

  • buttons in the add repeat dialog
  • horizontal and vertical view
  • long label of the repeat, null label of the repeat
  • big font sizes and a device with a smaller screen
  • LTR and RTL
  • light and dark mode

Separate issue filed #6981

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Redmi 9t "Do not add" repeat button

3 participants