Closed jauggy closed 2 months ago
@L-e-x-o-n can you please review if you have spare time thanks.
Should uncertainty arrow should always be next to number (not in next row)?
For reference, looks the same with and without PR, except grey uncertainty
Yeah your screenshot looks wrong. I'll take a look.
@L-e-x-o-n can you try in incognito mode. Maybe css is getting cached? Also please check you have the latest from this pr/branch. In my PR, each row should only take up one line and the arrow should be on same row.
Tried, on latest PR, same thing :( Firefox
I tested with Firefox again (latest version 125.0.2 and it works). can you do an experiment for me?
Open up
assets/scss/custom/_styles.scss
and modify to add background-color to the Map column:
Find this section
//Map column for Ratings table
.table-ratings tr th:nth-child(1) {
width: 200px;
}
Then modify to this:
//Map column for Ratings table
.table-ratings tr th:nth-child(1) {
width: 200px;
background-color: red;
}
Do you get the new color for the map heading?
I tested with Firefox again (latest version 125.0.2 and it works). can you do an experiment for me?
Open up
assets/scss/custom/_styles.scss
and modify to add background-color to the Map column: Find this section//Map column for Ratings table .table-ratings tr th:nth-child(1) { width: 200px; }
Then modify to this:
//Map column for Ratings table .table-ratings tr th:nth-child(1) { width: 200px; background-color: red; }
Do you get the new color for the map heading?
Doesn't work, even with cleared cache...
Part of the local setup is sass install
mix sass.install
@L-e-x-o-n Can you run that and try again?
Part of the local setup is sass install
mix sass.install
@L-e-x-o-n Can you run that and try again?
No changed :( Would be nice if someone else can try, maybe it's just me?
geekingfrog can you help test this PR? If the css is working correctly you should be able to go here:
http://localhost:4000/battle/ratings
and the columns will all be same width except the first one. For some reason it's not working for Lexon.
EDIT: Nevermind geekingfrog; Adam has tested it below.
I can confirm the PR works as expected:
On master:
On your branch:
This is working as expected.
Thanks Adam. I wonder why css is not working on Lexon's machine.
It breaks down under 1080px for viewport width, but otherwise it's a net improvement, can fix that (mobile view?) later.
Currently in production it looks like this for small screens:![1](https://github.com/beyond-all-reason/teiserver/assets/1305569/23465577-8f36-4623-8b61-14d9a7f1372f)
This ticket gives a better layout for smaller screens by adjusting the column widths and padding. Also I changed the uncertainty delta color to grey because it will always go down and users might be confused why that column is always red (and it's not bad to go down).![2](https://github.com/beyond-all-reason/teiserver/assets/1305569/31cde8c3-f0bb-4ccf-9032-5b1e0552bbfc)
Test Steps
Go here http://localhost:4000/battle/ratings
Test on resolutions such as 1360 x 768 1280 x 1024 (or similar if you don't have any of these screen sizes)