Skip to content

Various fixes #275

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

Merged
merged 12 commits into from
Jun 9, 2025
Merged

Various fixes #275

merged 12 commits into from
Jun 9, 2025

Conversation

frontsideair
Copy link
Collaborator

@frontsideair frontsideair commented Jun 9, 2025

  • Add word-break to long headings to prevent overflow
  • Reduce padding in mobile view for above the fold
  • Add gap between cta and hero image
  • Fix featured blog on mobile styles
  • Fix featured blog image aspect and alignment

Before

recording1.mov

After

recording2.mov

Copy link

github-actions bot commented Jun 9, 2025

Preview for aba3669

  • For faster build, the doc pages are not included in the preview.
  • Redirects are disabled in the preview.
Open in StackBlitz

Copy link
Collaborator

@daniellok-db daniellok-db left a comment

Choose a reason for hiding this comment

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

this definitely improves the experience, but i feel like we should probably also reduce the header text size on tiny screens

@frontsideair
Copy link
Collaborator Author

Added XS and XXS breakpoints, and each heading now goes one down per breakpoint. This is on Pixel 4, one of the narrowest devices in Chrome Devtools.

Screenshot 2025-06-09 at 13 15 21

@frontsideair
Copy link
Collaborator Author

frontsideair commented Jun 9, 2025

@daniellok-db implemented link colors as discussed on Slack, only for markdown content so blog and releases.

Screenshot 2025-06-09 at 15 49 45

@frontsideair
Copy link
Collaborator Author

I tweaked things around a bit, still not 100% sure but it's a start at least. @daniellok-db

Header Header
Screenshot 2025-06-09 at 16 49 23 Screenshot 2025-06-09 at 16 50 43
Screenshot 2025-06-09 at 16 49 46 Screenshot 2025-06-09 at 16 50 59

Copy link
Collaborator

@TomeHirata TomeHirata left a comment

Choose a reason for hiding this comment

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

LGTM

@TomeHirata TomeHirata merged commit fa64325 into mlflow:main Jun 9, 2025
4 checks passed
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.

4 participants