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

Per question custom style #2636

Merged

Conversation

santosh-pingle
Copy link
Collaborator

@santosh-pingle santosh-pingle commented Jul 24, 2024

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).

Fixes #[issue number]

Description
Add implementation to customize the style for question item. Allow developers to provide custom style names through the questionnaire JSON file. If a question item with a custom style is reused from the item pool, then apply the default style

Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?

Type
Choose one: Feature

Screenshots (if applicable)

Checklist

  • I have read and acknowledged the Code of conduct.
  • I have read the Contributing page.
  • I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
  • I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
  • I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
  • I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
  • I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

Copy link
Collaborator

@FikriMilano FikriMilano left a comment

Choose a reason for hiding this comment

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

@santosh-pingle Thanks for the PR!

@santosh-pingle santosh-pingle changed the title In Progress : Per question custom style Per question custom style Aug 7, 2024
@santosh-pingle santosh-pingle marked this pull request as ready for review August 7, 2024 07:21
@santosh-pingle
Copy link
Collaborator Author

Screenshot 2024-08-07 at 5 53 32 PM Screenshot 2024-08-07 at 5 55 46 PM

@santosh-pingle
Copy link
Collaborator Author

Screenshot 2024-08-13 at 5 16 32 PM Screenshot 2024-08-13 at 5 17 41 PM

@santosh-pingle
Copy link
Collaborator Author

Screenshot 2024-08-21 at 12 21 57 PM

@santosh-pingle santosh-pingle requested a review from a team as a code owner August 21, 2024 07:32
Copy link
Collaborator

@FikriMilano FikriMilano left a comment

Choose a reason for hiding this comment

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

Looks good! Just a very small comment

@santosh-pingle santosh-pingle enabled auto-merge (squash) September 4, 2024 12:18
@santosh-pingle santosh-pingle merged commit d9653b6 into google:master Sep 4, 2024
6 checks passed
ndegwamartin pushed a commit to opensrp/android-fhir that referenced this pull request Sep 10, 2024
* per question custom style

* adding missing file.

* update default style.

* textAppearance support

* rename custom attributes.

* github documentation for custom style example.

* Address review comments.

* address review comment.

* update text format icon as component icon.

* code cleanup.

* Code refactoring and cleanup.

* custom style example with multiple question items.

* Address review comments.

* support prefix per question item custom style

* Revert dataconfig changes for custom style mapping.

* Address review comments.

* Address review comments.

* Address review comments.

* Address review comments.

* Address review comment.

---------

Co-authored-by: Santosh Pingle <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Complete
Development

Successfully merging this pull request may close these issues.

5 participants