Skip to content

Commit

Permalink
Removes the scrollbar from the 'Player setup' menu (#5049)
Browse files Browse the repository at this point in the history
# About the pull request

Removes the scrollbar from the pre-roundstart 'Player setup' menu by
increasing its height slightly.

# Explain why it's good for the game

There's nowhere to actually scroll to, so it's just taking up a big
chunk of the space on the right side of the window.


# Testing Photographs and Procedure
<details>
<summary>Screenshots & Videos</summary>

**Before:**
![old
1](https://github.com/cmss13-devs/cmss13/assets/57483089/bcf463ed-390e-41df-91b4-29d65d6dd943)

**After:**
![new
1](https://github.com/cmss13-devs/cmss13/assets/57483089/ca75110c-4a0c-4ca1-8251-d4c1c981b263)

</details>


# Changelog
:cl:
ui: Removed the scrollbar from the 'Player setup' menu.
/:cl:
  • Loading branch information
SabreML authored Dec 3, 2023
1 parent c46d754 commit 7410dd9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion code/modules/mob/new_player/new_player.dm
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@
output += "</div>"
if (refresh)
close_browser(src, "playersetup")
show_browser(src, output, null, "playersetup", "size=240x[round_start ? 330 : 460];can_close=0;can_minimize=0")
show_browser(src, output, null, "playersetup", "size=240x[round_start ? 360 : 460];can_close=0;can_minimize=0")
return

/mob/new_player/Topic(href, href_list[])
Expand Down

0 comments on commit 7410dd9

Please sign in to comment.