-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Room UI Redesign: Migrate Modals #3363
Merged
Merged
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
cff506e
Reduce custom CSS
robertlong a5aa8ed
Migrate avatar/scene/object modals
robertlong 18185fc
Fix ObjectUrlModal
robertlong 1447fab
Fix bugs with media-browser modals
robertlong 0a19c2f
Migrate client-info-dialog
robertlong d3ca01b
Remove console.log
robertlong 850e2cb
Migrate CloseRoomModal
robertlong 8bf3f27
Remove unused ProfileInfoHeader component
robertlong 8de25bd
Remove reference to CloseRoomDialog
robertlong ba564b6
Add CloseRoomModal story
robertlong bdb8fed
Migrate to WebVRUnsupportedModal
robertlong ba9d0d9
Remove unused WebRTCScreenShareUnsupported dialog
robertlong 29b6f4f
Migrate Tweet Modal
robertlong 749a2a1
Revert tweet-media-button changes
robertlong 7c6b7ec
Migrate OAuthDialog
robertlong 18ae189
Fix twitter oauth modal
robertlong d272e51
Throw an error instead of returning
robertlong dca904e
Change oauth check.
robertlong 6bdaaf5
Set doingTwitterOAuth
robertlong eb68c83
postMessage from the window not window.opener
robertlong 8db0b08
Use correct postmessage handler
robertlong e1d48eb
Update can tweet permission after connect
robertlong 19b1892
Add logging
robertlong 6aa6763
Clean up tweet handler
robertlong a32e1c2
Fix close tweet modal
robertlong 9876134
Remove console logs
robertlong 3ac6e8f
Fix pointer events on OAuthScreen
robertlong 2fdcdf5
Remove continueText
robertlong e64bbd3
Remove tweet-dialog
robertlong 3082d06
Remove console log
robertlong 9ade623
Fix Create Object Button Copy
robertlong File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: this was probably reverted in a merge from the master branch. So I'm updating it again in this PR. It's needed for the
useIntl
hook.