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

Remove game notes action tab #8925

Merged
merged 1 commit into from
Feb 28, 2021
Merged

Remove game notes action tab #8925

merged 1 commit into from
Feb 28, 2021

Conversation

DanVanAtta
Copy link
Member

@DanVanAtta DanVanAtta commented Feb 28, 2021

Several discussions have happened in forums, generall agreed that the
game notes in the action tabs is not particularly useful. To keep things
simpler, this update remove the game notes tab.

Release Notes

REMOVE|Removed game notes 'action' tab, game notes are now only available from the 'help > game notes' menu.

Several discussions have happened in forums, generall agreed that the
game notes in the action tabs is not particularly useful. To keep things
simpler, this update remove the game notes tab.
@beelee1
Copy link
Contributor

beelee1 commented Feb 28, 2021

I agree but just want to point out that you can't move the map when accessing the game notes from the help tab. It'd be nice if you could. Not a biggy but still a difference.

@codecov
Copy link

codecov bot commented Feb 28, 2021

Codecov Report

Merging #8925 (ab6e0c6) into master (1b1f976) will increase coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #8925   +/-   ##
=========================================
  Coverage     28.40%   28.40%           
- Complexity     8365     8366    +1     
=========================================
  Files          1300     1299    -1     
  Lines         80823    80817    -6     
  Branches      11039    11039           
=========================================
- Hits          22956    22955    -1     
+ Misses        55696    55693    -3     
+ Partials       2171     2169    -2     
Impacted Files Coverage Δ Complexity Δ
...n/java/games/strategy/triplea/ui/TripleAFrame.java 0.00% <ø> (ø) 0.00 <0.00> (ø)
...trategy/triplea/ui/menubar/help/GameNotesMenu.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...rc/main/java/games/strategy/net/nio/NioReader.java 82.50% <0.00%> (-3.75%) 17.00% <0.00%> (-1.00%)
.../src/main/java/games/strategy/net/nio/Decoder.java 75.86% <0.00%> (ø) 14.00% <0.00%> (ø%)
...strategy/net/nio/ServerQuarantineConversation.java 91.22% <0.00%> (+1.75%) 13.00% <0.00%> (+1.00%)
...lea/ai/flowfield/influence/InfluenceTerritory.java 100.00% <0.00%> (+3.03%) 14.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1b1f976...6bfc16d. Read the comment docs.

@Cernelius
Copy link
Contributor

I'm against this change unless it is also assured that the notes always stay above the map view. I'm referring to the following issue, which impacts the game notes as well as the unit help window.
#8607

Even in case that would be fixed, it has so far happened so frequently that windows have issues of various kind that I would suggest keeping this rather useless tab just as a fall-back for future issues with windows.

Besides, I think this is something which should be better discussed in forum (just ask users if they like reading notes in the tabs any better than via the menu).

@beelee1
Copy link
Contributor

beelee1 commented Feb 28, 2021

@Cernelius I think it was talked about in forum. i had meant to comment but forgot about it until seeing it here.

edit
Yea #8607 makes a good point. i wasn't aware of that

@DanVanAtta
Copy link
Member Author

@Cernelius I think it was talked about in forum. i had meant to comment but forgot about it until seeing it here.

Yes, it was. I'm not sure where the thread is, the consensus between several folks was that it's not useful.

@DanVanAtta
Copy link
Member Author

Fixing the always on top seems easy enough actually, will follow up.

@DanVanAtta DanVanAtta merged commit 51b2a42 into master Feb 28, 2021
@DanVanAtta DanVanAtta deleted the remove-notes-action-tab branch February 28, 2021 02:35
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