Skip to content

Fix slideshow controls alignment#1

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

Fix slideshow controls alignment#1
hcooper merged 1 commit intoRopeWiki: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
Copy link
Copy Markdown
Collaborator

hcooper commented Apr 3, 2026

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