-
Notifications
You must be signed in to change notification settings - Fork 2
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
#772: Incorporate hamburger menu in web view component #793
Conversation
… hooked it up to be used in the toolbar and platform-tab-title.component.
…containerRef to determine position of menu. Added some "dummy" web view menu data for resourceViewer.react Modified createRCDockTabFromTabInfo to pass web view type to the title and made the title's icon into a menu button (instead of the hamburger icon).
ae68b8a
to
aa4c3dd
Compare
…n-webviews # Conflicts: # lib/platform-bible-react/dist/index.cjs # lib/platform-bible-react/dist/index.cjs.map # lib/platform-bible-react/dist/index.js # lib/platform-bible-react/dist/index.js.map # lib/platform-bible-react/src/components/toolbar.component.tsx # src/extension-host/data/menu.data.json REVIEW: Had to change the name of the resource viewer extension to use camelCase (resource-viewer => resourceViewer) in the manifest. TJ and Matt need to discuss this further.
…dded positioning logic based on containerRef (eliminating the need for an offset-based approach. Also fixed layout of icon in webview title bar.
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.
thanks for the great work on this! How exciting!! :D
Reviewed 1 of 12 files at r1, 16 of 16 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved
# Conflicts: # lib/platform-bible-react/dist/index.cjs # lib/platform-bible-react/dist/index.cjs.map # lib/platform-bible-react/dist/index.js # lib/platform-bible-react/dist/index.js.map
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.
Reviewed 2 of 2 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved
… hooked it up to be used in the toolbar and platform-tab-title.component.
This change is