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

UI scroll issue #1074

Merged
merged 3 commits into from
Mar 24, 2025
Merged

UI scroll issue #1074

merged 3 commits into from
Mar 24, 2025

Conversation

sushant-suse
Copy link
Contributor

@sushant-suse sushant-suse commented Mar 21, 2025

Which issue(s) this PR fixes:

Issue longhorn/longhorn#8143

What this PR does / why we need it:

It modifies two things:

  1. Initially whenever there was large code block, it gave a long horizontal scroll. I modified the config.toml file and added a flag that made hugo to use html and css with the markdown. Apart from this, I added 2 classes in assets/style.sass to modify the width of the code block and added line-wrap to it.
  2. The default page was having a lot of empty white space so I increased the width of the actual content and reduced the size of the Recent Articles recommendation.

Special notes for your reviewer:

This affects all code blocks in all documentation pages.
This affects the page structure of all the knowledge base articles.

Additional documentation or context

No

sushant-suse added 2 commits March 21, 2025 13:03
@sushant-suse sushant-suse requested a review from a team as a code owner March 21, 2025 08:29
Copy link

netlify bot commented Mar 21, 2025

Deploy Preview for longhornio ready!

Name Link
🔨 Latest commit 44e2e9d
🔍 Latest deploy log https://app.netlify.com/sites/longhornio/deploys/67e1114884153f000852785b
😎 Deploy Preview https://deploy-preview-1074--longhornio.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: sushant-suse <sushant.gaurav@suse.com>
Copy link
Contributor

@houhoucoop houhoucoop left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@derekbit derekbit left a comment

Choose a reason for hiding this comment

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

LGTM

@derekbit derekbit merged commit 6aeae9f into longhorn:master Mar 24, 2025
6 checks passed
@sushant-suse sushant-suse deleted the ui-scroll-issue branch March 24, 2025 08:19
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.

3 participants