Skip to content

Added showPillIcon() method to SettingsListItem#7724

Merged
nalcalag merged 2 commits intodevelopfrom
feature/noelia/show_pill_icon_method
Feb 12, 2026
Merged

Added showPillIcon() method to SettingsListItem#7724
nalcalag merged 2 commits intodevelopfrom
feature/noelia/show_pill_icon_method

Conversation

@nalcalag
Copy link
Contributor

@nalcalag nalcalag commented Feb 11, 2026

Task/Issue URL: https://app.asana.com/1/137249556945/task/1213217874770243

Description

Implement showPillIcon method to set visibility programatically in SettingsListItem

Steps to test this PR

  • Install from branch
  • Go to Settings > ADS Preview
  • Tap on List items
  • Check pill icons on Settings items look correct

No UI changes


Note

Low Risk
Small UI-component API addition and preview wiring change; risk is limited to pill visibility/attribute behavior in settings list items.

Overview
SettingsListItem now exposes showPillIcon(Boolean) to control the yellow pill visibility programmatically, and initialization was refactored to always set pillText while delegating visibility to the new method.

The internal design-system preview for the settings list item was updated to default pillIcon to false in XML and then explicitly enable it in SettingsListItemComponentViewHolder.bind to demonstrate runtime toggling.

Written by Cursor Bugbot for commit 1884783. This will update automatically on new commits. Configure here.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@nalcalag nalcalag force-pushed the feature/noelia/show_pill_icon_method branch from 33b6dd1 to 1884783 Compare February 11, 2026 23:48
@nalcalag nalcalag marked this pull request as ready for review February 11, 2026 23:49
@nalcalag nalcalag merged commit e874403 into develop Feb 12, 2026
26 of 27 checks passed
@nalcalag nalcalag deleted the feature/noelia/show_pill_icon_method branch February 12, 2026 10:32
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.

2 participants