Skip to content

Fix slideshow controls alignment#1

Merged
hcooper merged 1 commit into
RopeWiki:mainfrom
MattNotarangelo:fix/slideshow-controls-alignment
Apr 3, 2026
Merged

Fix slideshow controls alignment#1
hcooper merged 1 commit into
RopeWiki:mainfrom
MattNotarangelo:fix/slideshow-controls-alignment

Conversation

@MattNotarangelo

Copy link
Copy Markdown
Contributor

Summary

Moved from RopeWiki/commonjs#16 per review feedback — CSS belongs here, not injected via JS.

  • Center page number readout by adding left: 0 (overrides default -0.3em offset)
  • Align slider with pause button by zeroing asymmetric vertical margins (-11px top, 7px bottom → 0)
  • Round all corners of pause button evenly (border-radius: 4px) instead of only the right side

Test plan

Console command to preview all three fixes:

var s = document.createElement('style');
s.innerHTML = '.srf-slideshow .slideshow-nav-readout { left: 0; } .srf-slideshow .slideshow-nav { margin-top: 0; margin-bottom: 0; } .srf-slideshow .slideshow-nav-wrapper .ui-button-icon-only { border-radius: 4px; }';
document.body.appendChild(s);

- Center page number readout by removing left offset
- Align slider with pause button by zeroing asymmetric vertical margins
- Round all corners of pause button instead of right side only
@hcooper

hcooper commented Apr 3, 2026

Copy link
Copy Markdown
Collaborator

I've pushed this to production - looks good.

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.

2 participants