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

Master #941

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Master #941

wants to merge 4 commits into from

Conversation

clevengr
Copy link
Contributor

Description of what the PR does

Updates the WTI Team Guide (.docx and .pdf) so that screenshots match the V9.10 distribution. Also adds a section on "Scoreboard" (since no such menu item existed previously), along with an example of "popup notifications" which now require dismissal.

Updates all screenshots to show "Team XX" in the header, to show the "Scoreboard" menu item, and to show the updated "copyright" dates in the footer.

(Note: I did not actually generate completely new screenshots for all cases, since several of them require (or at least, should contain) complex sets of contest data (problems, scores, etc.). Rather, I cheated: I did "screen grabs" of the missing elements (such as "Team XX" and "Copyright 2020-2024") and added them to the top of existing images. If someone can show me where this makes a difference, I'll go back and do it the long, hard way...)

Issue which the PR addresses

Partial fix for #929.

Environment in which the PR was developed (OS,IDE, Java version, etc.)

Windows 11, Chrome Version 120.0.6099.131 (Official Build) (64-bit)

Precise steps for testing the PR (i.e., how to demonstrate that it works correctly)

Read the WTI Team Guide, verify that (1) every menu item is explained, and (2) all screenshots display the appropriate data elements.

@clevengr clevengr added this to the 9.10.0 milestone Mar 16, 2024
Copy link
Contributor

@troy2914 troy2914 left a comment

Choose a reason for hiding this comment

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

seems good enough to me.

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.

2 participants