Skip to content
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

Add devtools #19

Merged
merged 2 commits into from
Mar 3, 2025
Merged

Add devtools #19

merged 2 commits into from
Mar 3, 2025

Conversation

corinagum
Copy link
Contributor

@corinagum corinagum commented Mar 3, 2025

Devtools

  • Use FluentUI
  • Use fluentui icon fonts instead of svg's
  • Componentize:
    • Chats
    • Chat container/message container/
    • Markdown in messages
    • ComposeBox
    • (New Message Toolbar)
    • MessageReactions toolbar
    • Page navigation
    • Devtools banner with beta badge
    • Typing indicator
    • Activities screen / details
    • Attachments
  • Redesigned:
    • Streaming
    • Scrollbar
    • LightTheme/DarkTheme (Dark theme is default unless the system prefers light theme)
  • Hooks
    • useSparkApi
    • useTheme

Notes

  • Keyboard navigation semi-works, but is not 100% for chat messages.
  • Generally tried to pay attention to a11y while developing, but it will get more of my focus next month
  • Still need to implement Ctrl+Shift for new lines in ComposeBox
  • Still need to implement up arrow chat message history

Screenshots

Offline UI
offline UI image

Online UI + streaming
online ui with streaming image

Message reactions
message reactions image

Activities view:
image

No-activities Activities view:
image

Activity details:
image

Cards screen:
image

Attachments:
image

@aacebo aacebo merged commit a16e0cf into main Mar 3, 2025
5 checks passed
@aacebo aacebo deleted the cg/devtools branch March 3, 2025 23:07
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