Skip to content

PR: Fixing Github Copilot OAuth Dialog#45

Open
YoruAkio wants to merge 2 commits intoItsWendell:mainfrom
YoruAkio:akio/fix-copilot-auth-ui
Open

PR: Fixing Github Copilot OAuth Dialog#45
YoruAkio wants to merge 2 commits intoItsWendell:mainfrom
YoruAkio:akio/fix-copilot-auth-ui

Conversation

@YoruAkio
Copy link
Copy Markdown
Contributor

Making the ui the same as the openai oauth ( headless )

Before:
CleanShot 2026-02-19 at 04 24 58

After:
CleanShot 2026-02-19 at 04 23 02

- add proper timeout cleanup to prevent memory leaks on unmount
- enhance device code extraction with contextual regex matching
- simplify device code UI to inline layout
- handle both device code and auth URL in copy functionality
Copy link
Copy Markdown
Owner

@ItsWendell ItsWendell left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! Left a few minor suggestions, nothing blocking. Main one is keeping the device code visually distinct in the new compact layout so it's easy for users to read and type.

- update device code UI with code styling
- refine regex for device code extraction
@YoruAkio
Copy link
Copy Markdown
Contributor Author

Updated UI Preview.

CleanShot 2026-02-19 at 22 10 26

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