Skip to content

fixed unresponsiveness upon closing Developer logs#249

Merged
its-me-abhishek merged 4 commits intoCCExtractor:mainfrom
Ankit-69k:fix/developerLogsDialog
Dec 6, 2025
Merged

fixed unresponsiveness upon closing Developer logs#249
its-me-abhishek merged 4 commits intoCCExtractor:mainfrom
Ankit-69k:fix/developerLogsDialog

Conversation

@Ankit-69k
Copy link
Copy Markdown
Contributor

Description

This PR fixes the issue regarding the unresponsiveness upon closing Developer logs.
Actually it was not an issue with the Developer logs but with nested Dialog components, they were not properly used.

Checklist

  • Ran npx prettier --write . (for formatting)
  • Ran gofmt -w . (for Go backend)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 6, 2025

Thank you for opening this PR!

Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools.

Please take a moment to:

  • Check the "Files changed" tab
  • Leave comments on any lines for functions, comments, etc. that are important, non-obvious, or may need attention
  • Clarify decisions you made or areas you might be unsure about and/or any future updates being considered.
  • Finally, submit all the comments!

This helps make the review process smoother and gives us a clearer understanding of your thought process.

Once you've added your self-review, we'll continue from our side. Thank you!

@its-me-abhishek its-me-abhishek merged commit b2cd963 into CCExtractor:main Dec 6, 2025
4 checks passed
ShivaGupta-14 pushed a commit to ShivaGupta-14/ccsync that referenced this pull request Dec 25, 2025
* fixed unresponsiveness upon closing Developer logs

* test: remove `onOpenChange` prop from `DevLogs` component renders in tests.

* refactor: Update DevLogs tests by removing Dialog mocks, enhancing Select mocks, and adjusting snapshots for DevLogs as a content component.

* fixed movile UI for the Developer logs
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.

Screen becomes unresponsive upon closing Developer logs

2 participants