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

properly display popup content #711

Merged
merged 3 commits into from
Jun 10, 2024
Merged

properly display popup content #711

merged 3 commits into from
Jun 10, 2024

Conversation

mruwnik
Copy link
Collaborator

@mruwnik mruwnik commented Jun 8, 2024

No description provided.

'6306', // What is an intelligence explosion? debugger eval code:1:26
'5943', // How might AGI kill people? debugger eval code:1:26
'7748', // What is a "warning shot"?
]
Copy link
Contributor

Choose a reason for hiding this comment

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

big fan

@zarSou9
Copy link
Contributor

zarSou9 commented Jun 8, 2024

Code looks fire, and seems to be working. But just to make sure @melissasamworth, you still don't want the loading... to be in a big box? If so, I think we should at least remove the feedback buttons while it is loading.
image

Also, Nemo, I got this response to a question, I'm guessing there are unwritten questions where there shouldn't be in coda.
image

@zarSou9
Copy link
Contributor

zarSou9 commented Jun 8, 2024

Oh wait, there is also a bug, clicking on the settings causes the questions to reload. It actually might be nice to also have a refresh questions button come to think of it

Screen.Recording.2024-06-08.at.10.20.09.AM.mov

@mruwnik
Copy link
Collaborator Author

mruwnik commented Jun 8, 2024

good catch with the bug - @melissasamworth what do you think about it?

@zarSou9
Copy link
Contributor

zarSou9 commented Jun 10, 2024

@mruwnik Let me know when you've fixed the bug and resolved conflicts so I can merge. Also my humble opinion is to keep the big loading box but up to you

@mruwnik
Copy link
Collaborator Author

mruwnik commented Jun 10, 2024

done

@zarSou9 zarSou9 merged commit 6593238 into stampy-redesign Jun 10, 2024
1 check passed
@zarSou9 zarSou9 deleted the sundry-fixes branch June 10, 2024 22:33
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