Skip to content

feat: remove tertiary color from expandable list item#18

Merged
Linux123123 merged 1 commit intomainfrom
fix/incorrect-color-on-card
Mar 26, 2026
Merged

feat: remove tertiary color from expandable list item#18
Linux123123 merged 1 commit intomainfrom
fix/incorrect-color-on-card

Conversation

@Nirmanas
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@kvietelaitis kvietelaitis left a comment

Choose a reason for hiding this comment

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

gg humongous

@Linux123123 Linux123123 merged commit 9f89047 into main Mar 26, 2026
3 checks passed
@Linux123123 Linux123123 deleted the fix/incorrect-color-on-card branch March 26, 2026 19:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the explicit tertiary background color override from the expandable list items on the document success screen, allowing them to fall back to the shared WrapCard/Material defaults.

Changes:

  • Removed the colors = CardDefaults.cardColors(containerColor = MaterialTheme.colorScheme.tertiary) override from WrapExpandableListItem usage.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 183 to 186
hideSensitiveContent = false,
collapsedMainContentVerticalPadding = SPACING_MEDIUM.dp,
expandedMainContentVerticalPadding = SPACING_MEDIUM.dp,
colors =
CardDefaults.cardColors(
containerColor = MaterialTheme.colorScheme.tertiary,
),
)
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

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

After removing the custom colors parameter, CardDefaults and MaterialTheme imports in this file are now unused. Please delete those imports to avoid unused-import warnings/failing static checks.

Copilot uses AI. Check for mistakes.
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.

4 participants