Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Left justify data and labels, remove '--' and align data start with label start for easier readability and cleaner looks #644

Merged
merged 9 commits into from
Jan 7, 2024

Conversation

barjac
Copy link

@barjac barjac commented Jan 5, 2024

This PR attempts to improve the readability and looks of the internal reporter by left aligning all the data in each column rather than centring it within the current column widths which looks ragged.

It makes sure that the start of each data item is below the start of the column label.

It removes the '--' items which indicate that there is no data. However we can see that there is no data if there is nothing there.
I can maybe understand using this in a purely testing environment but not for production as it is serving no purpose.

I will add a screen shot later.

@barjac
Copy link
Author

barjac commented Jan 5, 2024

Screenshot_20240105_095612

@tmiw
Copy link
Collaborator

tmiw commented Jan 5, 2024

Thanks for creating this PR! Were you able to do any experimentation to see if it were possible to keep the column titles center aligned while left aligning everything else? I remember trying before and being unable to.

@barjac
Copy link
Author

barjac commented Jan 5, 2024

Thanks for creating this PR! Were you able to do any experimentation to see if it were possible to keep the column titles center aligned while left aligning everything else? I remember trying before and being unable to.

I read some wx docs and could see nothing related, however I'm not familiar with wxGTK capabilities but I really don't think that would be better.
Having the label accurately aligned with the data does not look bad at all. If the data includes narrow items as in the Msg column alongside the odd wide one, then a centred label could be very misaligned with the narrow items.
I used this in our daily 60m net today and took the screen shot below.
We have devised a new use for the Msg column during nets ;) Each joining member of the group adds his number to the Msg as he joins the net. By sorting on Msg the net stays in sequence. #3 was not reporting but it still worked well. Adding comments after the number still works fine and with left justification the numbers stay lined up.

Screenshot_20240105_154623c

@Tyrbiter
Copy link

Tyrbiter commented Jan 5, 2024

I'm not against this, however I think it may not be ideal for the distance and SNR columns, I think that they look better if right-justified. Perhaps for distance the own station value should be 0 or blank instead of 0.0 as well.

@barjac
Copy link
Author

barjac commented Jan 5, 2024

I'm not against this, however I think it may not be ideal for the distance and SNR columns, I think that they look better if right-justified. Perhaps for distance the own station value should be 0 or blank instead of 0.0 as well.

Yes I agree, seems odd that 0 is displayed as 0.0.

@tmiw
Copy link
Collaborator

tmiw commented Jan 6, 2024

I was doing a bit more reading on UX standards for tables and they seem to have the following in common:

  • Text: left aligned
  • Numbers: right aligned
  • Column header alignment should match data

Example guidance: https://medium.com/mission-log/design-better-data-tables-430a30a00d8c

With that in mind, I pushed some changes to right-align "km/mi", "kHz/MHz", and "SNR" columns. It actually significantly improved it for me to the point where I think it might be counterproductive to center align the column headings (assuming that's actually possible without significant changes to wxWidgets itself). Hopefully that's okay for you guys too, but if not, we can revert and just have everything left aligned.

@barjac
Copy link
Author

barjac commented Jan 6, 2024

I did a similar thing today before reading your last above. I also removed the check for distance < 10.0 which solved the 0.0 issue for me. I'm getting a bit lost in github - I don't see your changes :(
I also left the dates centred but maybe best not to do that now? I can't decide.

Screenshot_20240106_124107

@Tyrbiter
Copy link

Tyrbiter commented Jan 6, 2024

I've just made the changes as suggested, with dates centre-justified but then other text left-justified and numbers right-justified.

Also removed the <10.0 distance units check and that's nicer IMHO.

Not completely sure about the centred dates, I think that it's probably quite a weak change and is probably font-dependent.

@tmiw
Copy link
Collaborator

tmiw commented Jan 6, 2024

Yeah, considering the width of the text in the date columns, I don't think there's much difference other than for the headers. I'd say to back out that particular change to be more consistent with the other text-based columns.

@barjac
Copy link
Author

barjac commented Jan 6, 2024

OK the dates are back to left justified and I am very happy with it now. Note I also changed mi to Miles which looks much better and in most cases causes no width overhead.

Screenshot_20240106_201857

@tmiw
Copy link
Collaborator

tmiw commented Jan 6, 2024

Note I also changed mi to Miles which looks much better and in most cases causes no width overhead.

Seems like we'd want to keep it consistent with both units, right? As in, either abbreviate both or use the full name for both.

Though I'm also fine just using "Miles" if that is indeed better.

@barjac
Copy link
Author

barjac commented Jan 6, 2024

Note I also changed mi to Miles which looks much better and in most cases causes no width overhead.

Seems like we'd want to keep it consistent with both units, right? As in, either abbreviate both or use the full name for both.

No not really! 'Kilometers' is too long, 'Miles' is not and if you only use one of them then you will not care! ;)

Though I'm also fine just using "Miles" if that is indeed better.

Looks much better to me, 'mi' looks like an SI unit that I never heard of and it's not a common abbreviation for mile. :)
EDIT Hmm.. seems that it may be according to 'the internet', but I have never heard of it before.

OT/
BTW one of the recent changes to do with re-sizing of windows seems to have fixed the issue where a reduced height main window that was touching the top of the screen always re-started centred in the Y axis. I must re-test by removing the 'jiggle' workaround :)

@barjac
Copy link
Author

barjac commented Jan 6, 2024

I was trying to figure out what was causing the 'Last update' to be pushed off my screen. It appears to be mainly down to someone using an eight character locator. Can this field be limited to 6 characters? ...and lower case conversion for the last two?
Also SNR column seems over wide as it only ever needs 4 characters.
Miles/km is also wider than needed as it only needs 5 digits, unless it's catering for interplanetary contacts :)
By manually adjusting these the whole table fits again.

Screenshot_20240106_224900

@tmiw
Copy link
Collaborator

tmiw commented Jan 7, 2024

I'll go ahead and merge, then work on the stuff mentioned in the very last comment above.

@tmiw tmiw merged commit bd57914 into drowe67:master Jan 7, 2024
@barjac barjac deleted the bcj-reporter-left-justify branch November 26, 2024 20:24
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.

3 participants