Skip to content

ref(feedback): add tooltip when delete is disabled #89773

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

Merged
merged 1 commit into from
Apr 16, 2025

Conversation

michellewzhang
Copy link
Member

closes #87018

the feedback delete button is disabled if event admin access does not exist. add a tooltip explaining why when it's disabled

large view:

SCR-20250416-jlao

medium view:

SCR-20250416-jmqs

small view:

SCR-20250416-jmkt

bulk:

SCR-20250416-jnva

@michellewzhang michellewzhang requested a review from a team as a code owner April 16, 2025 17:27
@@ -43,7 +43,6 @@ export default function useFeedbackActions({feedbackItem}: Props) {
projectIds: feedbackItem.project ? [feedbackItem.project.id] : [],
});
const onDelete = useDeleteFeedback([feedbackItem.id], projectId);
const hasDelete = organization.features.includes('issue-platform-deletion-ui');
Copy link
Member Author

Choose a reason for hiding this comment

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

this flag is rolled out to GA.

Copy link
Member Author

Choose a reason for hiding this comment

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

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 16, 2025
@@ -184,9 +187,11 @@ function SmallWidth({feedbackItem}: {feedbackItem: FeedbackIssue}) {
key: 'delete',
priority: 'danger' as const,
label: t('Delete'),
hidden: !hasDelete,
Copy link
Member

Choose a reason for hiding this comment

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

hidden? i should use this more...

@michellewzhang michellewzhang merged commit c0cf982 into master Apr 16, 2025
46 checks passed
@michellewzhang michellewzhang deleted the mz/add-tooltip-to-delete-fb branch April 16, 2025 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[User Feedback] Delete button on feedback details header is disabled, and there's no tooltip explaining why
2 participants