esitarski / CrossMgr

Cyclo Cross Management Application
MIT License
41 stars 21 forks source link

Option to hide CustomCategories in CrossMgr HTML output #65

Closed kimble4 closed 3 years ago

kimble4 commented 4 years ago

When a number of CustomCategories are in use, the HTML output becomes unwieldy, making it hard to keep rider's names on the screen at the same time as their times.

A typical HPV racing example can be seen here: http://www.bhpc.org.uk/Data/Sites/1/media/events/events20/02darley_moor/2020-08-31-darley-moor-2-round-2-r2-.html

I've mucked about with trivial post-processing of the HTML (find&replace to shorten the "CustomCategoryN" headings, for example) but this complicates the workflow and is only a minor improvement.

A better solution would be for the HTML to provide a tickybox option to show/hide the CustomCategory columns, similar to the way that hundredths of a second or lap times can be shown/hidden. This way the times will be easily visible on reasonable screen sizes, but the riders' categories can be revealed if the viewer is interested in them.

esitarski commented 3 years ago

There is new checkbox in the html: Show: Custom Categories. This is available in the current Development release. Can you install the latest version and validate that it meets the need? Thanks.

kimble4 commented 3 years ago

There's a tickbox, but it isn't working...

Looks like you've got a typo at line 5509:

function ToggleCustomCategories() {
    showCustomCategories = !showCustomerCategories;
    RefreshResultsTable();
}

If I edit the HTML file to fix that, it has the desired result.

esitarski commented 3 years ago

I figured it out (Customer --> Custom). Too much influence from my day job ;)

I made the fix and checked it in. A new build should be available in a few minutes.

On Wed, Sep 16, 2020 at 1:14 PM Kim Wall notifications@github.com wrote:

There's a tickbox, but it isn't working...

Looks like you've got a typo at line 5509:

function ToggleCustomCategories() { showCustomCategories = !showCustomerCategories; RefreshResultsTable(); }

If I edit the HTML to fix that, it has the desired result.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/esitarski/CrossMgr/issues/65#issuecomment-693543376, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABGXKMIKJDIDMTVC666UKLSGDW6LANCNFSM4QUUJ62Q .

--

Edward Sitarski

kimble4 commented 3 years ago

That's working nicely.

Thanks!