Skip to content

Issue #19905: Improve cached doc comments page rendering#19942

Merged
romani merged 1 commit into
checkstyle:masterfrom
gianmarcoschifone:mobile
May 25, 2026
Merged

Issue #19905: Improve cached doc comments page rendering#19942
romani merged 1 commit into
checkstyle:masterfrom
gianmarcoschifone:mobile

Conversation

@gianmarcoschifone

Copy link
Copy Markdown
Member

Issue #19905

@gianmarcoschifone

Copy link
Copy Markdown
Member Author

GitHub, generate website

@gianmarcoschifone gianmarcoschifone marked this pull request as draft May 23, 2026 10:51
@github-actions

Copy link
Copy Markdown
Contributor

@gianmarcoschifone

Copy link
Copy Markdown
Member Author

image

@gianmarcoschifone gianmarcoschifone marked this pull request as ready for review May 23, 2026 11:19
@gianmarcoschifone

Copy link
Copy Markdown
Member Author

@Zopsss @romani done

@Zopsss Zopsss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

items:

pre {
overflow-x: auto;
}
} No newline at end of file

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please add an empty line at the end of the file

Comment on lines +14 to +16
pre {
overflow-x: auto;
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why do we need overflow-x?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

because using this property the block of code gets a scrollbar, allowing horizontal scrolling. Instead of making the lines of code go to the next line

Comment on lines +30 to +32
pre {
overflow-x: auto;
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

we already have this property set outside of media query, we don't need to repeat it again here.

h6 {font-size: 1.1em;}

main {
max-width: 1600px;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

How does the text wrapping is working? I am assuming that you added max-width so the content can only expands maximum to 1600px, if the screen size is smaller then content adjusts itself to match the screen size.

This is just my assumptions, please correct me if I am wrong.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, this is just to make the appearance on desktop a bit better. Since I was adding css I thought of adding this too

@Zopsss

Zopsss commented May 24, 2026

Copy link
Copy Markdown
Member

Github, generate website

@github-actions

Copy link
Copy Markdown
Contributor

@Zopsss

Zopsss commented May 24, 2026

Copy link
Copy Markdown
Member

Website looks good to me. @romani please review

@romani romani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks a lot

@romani romani merged commit a1e0103 into checkstyle:master May 25, 2026
127 of 128 checks passed
@gianmarcoschifone gianmarcoschifone deleted the mobile branch May 27, 2026 15:46
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