Skip to content

chore(issue summary): More obvious re-summarize button on summary in drawer #91841

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 3 commits into from
May 16, 2025

Conversation

roaga
Copy link
Member

@roaga roaga commented May 16, 2025

Removes the ellipsis menu and puts a more obvious button to re-generate the summary when in the drawer and the event id doesn't match.
Screenshot 2025-05-16 at 2 52 17 PM

@roaga roaga requested a review from a team May 16, 2025 21:54
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 16, 2025
@trillville
Copy link
Contributor

Removes the ellipsis menu and puts a more obvious button to re-generate the summary when in the drawer and the event id doesn't match. Screenshot 2025-05-16 at 2 52 17 PM

the language feels slightly confusing to me, like it makes me think its just going to redo the summary of the same event - some ideas:

  • "summarize active event"
  • "summarize current event"

i wonder if we'd also want to have some unobtrusive link/ref to the event that it is summarizing as well?

@roaga
Copy link
Member Author

roaga commented May 16, 2025

Screenshot 2025-05-16 at 3 05 49 PM @trillville done

EDIT: actually i think the link to the original is unnecessary

@roaga roaga merged commit 2048098 into master May 16, 2025
41 checks passed
@roaga roaga deleted the issue-summary/more-obv-resummarize branch May 16, 2025 23:57
andrewshie-sentry pushed a commit that referenced this pull request May 20, 2025
…drawer (#91841)

Removes the ellipsis menu and puts a more obvious button to re-generate
the summary when in the drawer and the event id doesn't match.
<img width="726" alt="Screenshot 2025-05-16 at 2 52 17 PM"
src="https://github.com/user-attachments/assets/33a0655d-6928-4922-8801-41bfdfa4c25b"
/>
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.

2 participants