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

Fixed documentation panel width resets after resizing browser window #15454

Conversation

matter-q
Copy link
Contributor

@matter-q matter-q commented Aug 9, 2022

What

Closes #14845

How

  • Fixed documentation panel width resets after resizing browser window
  • Fixed documentation panel unnecessary horizontal scroll for some sources (Facebook Marketing etc.)
  • Fixed main view with trial banner (removed double scrolls for main view)

Loom

https://www.loom.com/share/cad886ef8ef742f9b1f2b20250ea46d4

Fixed documentation panel unnecessary horizontal scroll
Fixed main view with trial banner
@github-actions github-actions bot added area/platform issues related to the platform area/frontend Related to the Airbyte webapp labels Aug 9, 2022
@matter-q matter-q marked this pull request as ready for review August 9, 2022 14:50
@matter-q matter-q requested a review from a team as a code owner August 9, 2022 14:50
@edmundito
Copy link
Contributor

@matter-q we are working on the same thing! I was working on #15168 and started by cleaning up the components and then realized that we have a lot of overlap. I have isolated the common changes here: #15455. What I'm thinking is that these should be merged separately and this PR should resolve conflicts.

@matter-q
Copy link
Contributor Author

matter-q commented Aug 9, 2022

@matter-q we are working on the same thing! I was working on #15168 and started by cleaning up the components and then realized that we have a lot of overlap. I have isolated the common changes here: #15455. What I'm thinking is that these should be merged separately and this PR should resolve conflicts.

Yes, you can merge your changes and I will resolve conflicts in my PR

…r-resizing-browser-window

# Conflicts:
#	airbyte-webapp/src/components/DocumentationPanel/DocumentationPanel.module.scss
#	airbyte-webapp/src/components/DocumentationPanel/DocumentationPanel.tsx
#	airbyte-webapp/src/components/Markdown/Markdown.tsx
Copy link
Contributor

@dizel852 dizel852 left a comment

Choose a reason for hiding this comment

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

Tested locally - Chrome, Safari, Firefox
The issue is fixed.
Code - LGTM.

One thing I'd like to highlight - I'm not sure we can fix that, overwise the content can become unreadable, so let's leave leave it as is, at least for now:
image

@matter-q matter-q merged commit 4704486 into master Aug 11, 2022
@matter-q matter-q deleted the mark/documentation-panel-width-resets-after-resizing-browser-window branch August 11, 2022 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to the Airbyte webapp area/platform issues related to the platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation panel width resets after resizing browser window
3 participants