Skip to content

fix: Apply Minor changes to AppProjects Details Page and Tabs#159

Merged
keithchong merged 2 commits intoredhat-developer:mainfrom
aali309:MinorchangesToDetailsPage
Jan 26, 2026
Merged

fix: Apply Minor changes to AppProjects Details Page and Tabs#159
keithchong merged 2 commits intoredhat-developer:mainfrom
aali309:MinorchangesToDetailsPage

Conversation

@aali309
Copy link
Copy Markdown
Collaborator

@aali309 aali309 commented Jan 15, 2026

Fixes: GITOPS-8634

Help icon + ArgoCD Link on Allow/Deny, Roles and Sync Windows tabs

Screenshot 2026-01-15 at 3 15 06 PM Screenshot 2026-01-15 at 3 15 32 PM Screenshot 2026-01-15 at 3 16 28 PM

Tooltip under Roles tab with more description

Screenshot 2026-01-15 at 3 19 05 PM

Signed-off-by: Atif Ali <atali@redhat.com>
@openshift-ci openshift-ci bot requested review from keithchong and wtam2018 January 15, 2026 20:13
</div>
<div>
<strong>{t('Policy Effect')}:</strong>{' '}
{effect === 'allow' ? `✅ ${t('Allow')}` : `❌ ${t('Deny')}`}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Could we use fontawesome instead of embedding them into the source file?

eg. something like this:

<span className='fa fa-check-square' style={{ color: 'green' }}>

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Screenshot 2026-01-20 at 1 51 11 PM-8cc1eef5-cc1f-420f-b1aa-86519294a0de Screenshot 2026-01-20 at 1 52 15 PM-e0e9c572-e2a1-4414-82a9-2423ac9db5c4

I also fixed this issue on Allow/Deny Lists cards, long group names in the "Group" column were being truncated.

Copy link
Copy Markdown
Collaborator

@keithchong keithchong left a comment

Choose a reason for hiding this comment

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

LGTM

@keithchong keithchong merged commit 966fc7f into redhat-developer:main Jan 26, 2026
3 checks passed
@aali309 aali309 deleted the MinorchangesToDetailsPage branch January 27, 2026 18:11
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