Skip to content

Conversation

@Mpdreamz
Copy link
Member

@Mpdreamz Mpdreamz commented Apr 29, 2025

image

Separate PR to #1197 since this might be controversial 😸

  • Ensure the page indicator aligns hard left (no more padding). This makes the marging in between the two vertical borders and the content exactly the same.

  • Move edit / submit an issue back up again since this often gets lost and visually anchors them in the same place for each page.

@Mpdreamz Mpdreamz requested a review from a team as a code owner April 29, 2025 19:36
@Mpdreamz Mpdreamz added the fix label Apr 29, 2025
@Mpdreamz Mpdreamz self-assigned this Apr 29, 2025
@Mpdreamz Mpdreamz changed the title feature/limited width layout 3 Slight on this page margin adjustments Apr 29, 2025
@Mpdreamz Mpdreamz changed the base branch from main to feature/limited-width-layout-2 April 29, 2025 19:37
@Mpdreamz Mpdreamz merged commit 3240315 into feature/limited-width-layout-2 Apr 30, 2025
15 checks passed
@Mpdreamz Mpdreamz deleted the feature/limited-width-layout-3 branch April 30, 2025 09:20
Mpdreamz added a commit that referenced this pull request May 1, 2025
* Full width layout PoC

* Also set max width for other elements

* Fixes and fine-tuning

* Cleanup

* Fix isolated build header position

* Formatting

* Prettier

* Limited max width PoC

* Use common resolution width

* Dynamic spacing for max-width layout

* ensure margins are dynamic again

* fix css lint

* Slight on this page margin adjustments (#1198)

* feature/limited width layout 4 (#1201)

* fixate max-width around max content-width

* npm fix lint

* Update src/Elastic.Markdown/Slices/_Layout.cshtml

Co-authored-by: Jan Calanog <[email protected]>

---------

Co-authored-by: Jan Calanog <[email protected]>

---------

Co-authored-by: Jan Calanog <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants