Skip to content

Conversation

@ludovic35
Copy link
Member

Note: Please transform - [ ] into - (NA) in the description when things are not applicable

Related issues

(Orange-OpenSource/ouds-ios#1136)

Description

  • Use internal icon for negative, positive, info and warning status. Icon is requested for accent and neutral.
  • remove disabled status, but use View.disabled instead.

Motivation & Context

Types of change

  • Breaking change (fix or feature that would change existing functionality)

Previews

Checklist

Contribution

Accessibility

  • My change follows accessibility good practices

Design

  • My change respects the design guidelines of Orange Unified Design System

Development

  • My change follows the developer guide
  • I checked my changes do not add new SwiftLint warnings
  • I have added unit tests to cover my changes (optional)

Documentation

  • My change introduces changes to the documentation and/or I have updated the documentation accordingly

Checklist (for Core Team only)

  • The evolution have been tested and the project builds for iPhones and iPads
  • Code review has been done by reviewers according to CODEOWNERS file
  • Design review has been done
  • Accessibility review has been done
  • Q/A team has tested the evolution
  • Documentation has been updated if relevant
  • Internal files have been updated if relevant (like CONTRIBUTING, DEVELOP, THIRD_PARTY, CONTRIBUTORS, NOTICE)
  • CHANGELOG has been updated respecting keep a changelog rules and reference the issues
  • The wiki has been updated if needed (optional)

@ludovic35 ludovic35 requested a review from pylapp as a code owner November 12, 2025 21:42
@ludovic35 ludovic35 force-pushed the 1136-component-update-badge-to-use-internal-icons-for-some-status branch from d4b73db to d1193af Compare November 12, 2025 21:57
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.

1 participant