Skip to content

fixes the appearance of the start button - #16

Open
tomjemmett wants to merge 2 commits into
mainfrom
fix_start_button
Open

fixes the appearance of the start button#16
tomjemmett wants to merge 2 commits into
mainfrom
fix_start_button

Conversation

@tomjemmett

Copy link
Copy Markdown
Member

if the browser was resized the start button can exhibit strange behaviour. this change makes the button sit inside its own div and then be right aligned. this prevents the button from filling the whole width of the container

if the browser was resized the start button can exhibit strange behaviour. this change makes the button sit inside its own div and then be right aligned. this prevents the button from filling the whole width of the container
Copilot AI lite review requested due to automatic review settings August 18, 2026 15:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts the Shiny UI layout so the dynamically-rendered “Start” button stays right-aligned and doesn’t behave oddly (or appear to stretch) after browser resize events.

Changes:

  • Wraps the start_button uiOutput() in a right-aligned container in the UI.
  • Simplifies the server-side start_button rendering by removing the bslib::layout_columns() wrapper and returning only the <a> button element.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
R/app_ui.R Wraps uiOutput("start_button") in a right-aligned div to control layout on resize.
R/app_server.R Removes layout_columns() around the Start link so the UI can handle alignment without column hacks.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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