Skip to content

Add button to expand popups to full height - #316

Merged
ScriptRaccoon merged 1 commit into
mainfrom
popup-expansion
Aug 1, 2026
Merged

Add button to expand popups to full height#316
ScriptRaccoon merged 1 commit into
mainfrom
popup-expansion

Conversation

@ScriptRaccoon

@ScriptRaccoon ScriptRaccoon commented Aug 1, 2026

Copy link
Copy Markdown
Owner

This PR adds a button to the popups that expands them to the full page height. By default, they only occupy up to 50% of the page height. This is particularly useful when reading long proofs. Although scrolling is possible and may still be necessary for some very long proofs, expanding the popup often eliminates the need for it and makes the proof easier to see as a whole.

Minimized State


Expanded State


Thoughts

  • The popups are not expanded by default (which would also make this new feature redundant), since many proofs are short and should not distract too much from the rest of the page.
  • The expand button does not make much sense for short proofs, and one could argue that it should not be displayed in those cases. However, it is simpler and more consistent to always show it. Any calculation that determines when the button should be displayed would be brittle and probably not worth the effort.
  • This change goes in the opposite direction from Extract separate proof length script #238 and Various UI improvements #221. Perhaps long proofs outside of content pages are fine after all. In contrast, creating a content page whose sole purpose is to present a single proof introduces an unnecessary indirection for the user. For example, the proof that the Brauer group functor Br : FldAb is finitary is somewhat long, but it is not worth adding a separate page for it.

@ScriptRaccoon ScriptRaccoon changed the title Add button to expand the popup Add button to expand popups to full height Aug 1, 2026
@ScriptRaccoon
ScriptRaccoon merged commit 20b01fe into main Aug 1, 2026
2 checks passed
@ScriptRaccoon
ScriptRaccoon deleted the popup-expansion branch August 1, 2026 10:41
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.

1 participant