Skip to content

Conversation

@remyjette
Copy link
Collaborator

What is this fixing or adding?

Implements suggestions from the generation-suggestions channel to add a summary row to the Multiworld tracker, so games that aren't the big async can see total number of checks/worlds completed easily.

How was this tested?

Generated a one player game, made sure the footer matched exactly. Waited a few minutes to ensure it updated with the main row. Generated a two player game, made sure things added up, made sure that the footer row continued to update as items were found.

If this makes graphical changes, please attach screenshots.

@ThePhar ThePhar added is: enhancement Issues requesting new features or pull requests implementing new features. affects: webhost Issues/PRs that touch webhost and may need additional validation. labels Jul 11, 2023
@ThePhar ThePhar requested a review from LegendaryLinux July 11, 2023 15:19
@ThePhar
Copy link
Contributor

ThePhar commented Jul 11, 2023

Requested a review from Farrak for front end changes, but back end changes look fine. I give my approval for those.

@remyjette
Copy link
Collaborator Author

I just noticed in a game I'm running on my own webhost that the summary row is still there on the multiFactorioTracker, and is mis-aligned due to the additional columns in that tracker.

Presumably it's similarly off for the lttp multi tracker, so I need to look into this.

Copy link
Contributor

@LegendaryLinux LegendaryLinux left a comment

Choose a reason for hiding this comment

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

I updated the JS to include a docblock and remove as much jQuery as possible. I really hate jQuery.

Let me know when you get the factorio tracker sorted.

@remyjette
Copy link
Collaborator Author

I updated the JS to include a docblock and remove as much jQuery as possible. I really hate jQuery.

Let me know when you get the factorio tracker sorted.

Thanks!!

And I just updated it to not render the summary row for game-specific multi trackers.

@LegendaryLinux LegendaryLinux merged commit 9323f7d into ArchipelagoMW:main Aug 29, 2023
@remyjette remyjette deleted the tracker-summary-footer branch August 29, 2023 22:01
FlySniper pushed a commit to FlySniper/Archipelago that referenced this pull request Nov 14, 2023
)

* WebHost: Add a summary row to the Multiworld Tracker

Implements suggestions from the generation-suggestions channel:
- https://discord.com/channels/731205301247803413/1124186131911688262
- https://discord.com/channels/731205301247803413/1109513647274856518

* Improve secondsToHours function, and remove jQuery from footerCallback function.

* Don't show the summary row on game-specific multi trackers

---------

Co-authored-by: Chris Wilson <[email protected]>
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
)

* WebHost: Add a summary row to the Multiworld Tracker

Implements suggestions from the generation-suggestions channel:
- https://discord.com/channels/731205301247803413/1124186131911688262
- https://discord.com/channels/731205301247803413/1109513647274856518

* Improve secondsToHours function, and remove jQuery from footerCallback function.

* Don't show the summary row on game-specific multi trackers

---------

Co-authored-by: Chris Wilson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

affects: webhost Issues/PRs that touch webhost and may need additional validation. is: enhancement Issues requesting new features or pull requests implementing new features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants