Skip to content

Extract player profile loading chrome logic to a dedicated class#21604

Merged
PunkPun merged 2 commits intoOpenRA:bleedfrom
dragunoff:feature/player-profile-chrome-logic
Oct 15, 2024
Merged

Extract player profile loading chrome logic to a dedicated class#21604
PunkPun merged 2 commits intoOpenRA:bleedfrom
dragunoff:feature/player-profile-chrome-logic

Conversation

@dragunoff
Copy link
Copy Markdown
Contributor

@dragunoff dragunoff commented Oct 3, 2024

Reduce a bit the clutter in MainMenuLogic.cs and in PlayerProfileLogic.cs

@dragunoff dragunoff force-pushed the feature/player-profile-chrome-logic branch from e4c38a6 to 99001ae Compare October 3, 2024 09:32
Copy link
Copy Markdown
Member

@PunkPun PunkPun left a comment

Choose a reason for hiding this comment

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

rebase needed

@dragunoff dragunoff force-pushed the feature/player-profile-chrome-logic branch from 99001ae to f73043d Compare October 7, 2024 11:16
@dragunoff
Copy link
Copy Markdown
Contributor Author

Update: rebased and extracted RegisteredProfileTooltipLogic from PlayerProfileLogic.cs so only a single class remains there.

@PunkPun PunkPun merged commit 791595e into OpenRA:bleed Oct 15, 2024
@PunkPun
Copy link
Copy Markdown
Member

PunkPun commented Oct 15, 2024

changelog

@dragunoff dragunoff deleted the feature/player-profile-chrome-logic branch October 15, 2024 10:54
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