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

feat(content, port): UI, accessibility updates, tweaks, remade work #4712

Closed
wants to merge 6 commits into from

Conversation

Kenan2000
Copy link
Collaborator

This should hopefully work correctly now

Ports these pull requests

CleverRaven/Cataclysm-DDA#47584,
CleverRaven/Cataclysm-DDA#47585,
CleverRaven/Cataclysm-DDA#58609,
CleverRaven/Cataclysm-DDA#51447,
CleverRaven/Cataclysm-DDA#59035,
CleverRaven/Cataclysm-DDA#50892,
CleverRaven/Cataclysm-DDA#41605,
CleverRaven/Cataclysm-DDA#57361,

So from pull request https://CleverRaven/Cataclysm-DDA#41605 tweaks from only a single file, ui.h, have been ported, whereas all other pull requests were essentially ported as fully as possible. This depends on #4634, #4635, so it cannot be merged until these are merged

So, again, thanks to @Coolthulhu, @yay855, @AlecWhite, whereas @olanti-p helped to fix some initial errors, whereas @KheirFerrum has greatly helped with tidying, fixing keyboard format errors in #4635, but there are still some things to fix so it's a draft

It's very important that these pull requests, #4634, #4635 along with this would be merged in exactly this order, to avoid conflicts, to avoid errors with files, I will ask for review when all of them are ready to be merged

Coauthors have been linked properly via GitHub Desktop built-in function to do this work hopefully

Kenan2000 and others added 5 commits May 31, 2024 22:57
Co-Authored-By: Zhilkin Serg <ZhilkinSerg@users.noreply.github.com>
Co-Authored-By: ZeroInternalReflection <89038572+ZeroInternalReflection@users.noreply.github.com>
Co-Authored-By: Jianxiang Wang (王健翔) <qrox@sina.com>
Co-Authored-By: eltank <8000047+eltank@users.noreply.github.com>
Co-Authored-By: Brambor <13402666+Brambor@users.noreply.github.com>
Co-Authored-By: martinrhan <53336429+martinrhan@users.noreply.github.com>
Co-Authored-By: Mark Langsdorf <mlangsdo@redhat.com>
Co-Authored-By: BevapDin <5095435+bevapdin@users.noreply.github.com>
Co-Authored-By: KheirFerrum <102964889+KheirFerrum@users.noreply.github.com>
Co-Authored-By: Coolthulhu <Coolthulhu@gmail.com>
Co-Authored-By: Zhilkin Serg <ZhilkinSerg@users.noreply.github.com>
Co-Authored-By: ZeroInternalReflection <89038572+ZeroInternalReflection@users.noreply.github.com>
Co-Authored-By: Jianxiang Wang (王健翔) <qrox@sina.com>
Co-Authored-By: eltank <8000047+eltank@users.noreply.github.com>
Co-Authored-By: Brambor <13402666+Brambor@users.noreply.github.com>
Co-Authored-By: martinrhan <53336429+martinrhan@users.noreply.github.com>
Co-Authored-By: Mark Langsdorf <mlangsdo@redhat.com>
Co-Authored-By: BevapDin <5095435+bevapdin@users.noreply.github.com>
Co-Authored-By: KheirFerrum <102964889+KheirFerrum@users.noreply.github.com>
Co-Authored-By: Rob Kuijper <robkuijper@live.nl>
Co-Authored-By: Jason Fields <jasonfields4@gmail.com>
Co-Authored-By: Coolthulhu <Coolthulhu@gmail.com>
…m/cataclysmbnteam/Cataclysm-BN into UIaccessibility_portagain_update3

Co-Authored-By: Zhilkin Serg <ZhilkinSerg@users.noreply.github.com>
Co-Authored-By: ZeroInternalReflection <89038572+ZeroInternalReflection@users.noreply.github.com>
Co-Authored-By: Jianxiang Wang (王健翔) <qrox@sina.com>
Co-Authored-By: eltank <8000047+eltank@users.noreply.github.com>
Co-Authored-By: Brambor <13402666+Brambor@users.noreply.github.com>
Co-Authored-By: martinrhan <53336429+martinrhan@users.noreply.github.com>
Co-Authored-By: Mark Langsdorf <mlangsdo@redhat.com>
Co-Authored-By: BevapDin <5095435+bevapdin@users.noreply.github.com>
Co-Authored-By: KheirFerrum <102964889+KheirFerrum@users.noreply.github.com>
Co-Authored-By: Rob Kuijper <robkuijper@live.nl>
Co-Authored-By: Jason Fields <jasonfields4@gmail.com>
Co-Authored-By: Coolthulhu <Coolthulhu@gmail.com>
Co-Authored-By: Olanti <olanti-p@yandex.ru>
Co-Authored-By: Chorus System <3629845+yay855@users.noreply.github.com>
@github-actions github-actions bot added docs PRs releated to docs page src PR changes related to source code. data PRs related to datas. Won't crash game (probably) labels May 31, 2024
Copy link
Contributor

autofix-ci bot commented May 31, 2024

Autofix has formatted code style violation in this PR.

I edit commits locally (e.g: git, github desktop) and want to keep autofix
  1. Run git pull. this will merge the automated commit into your local copy of the PR branch.
  2. Continue working.
I do not want the automated commit
  1. Format your code locally, then commit it.
  2. Run git push --force to force push your branch. This will overwrite the automated commit on remote with your local one.
  3. Continue working.

If you don't do this, your following commits will be based on the old commit, and cause MERGE CONFLICT.

@Kenan2000 Kenan2000 changed the title (feat, port): UI, accessibility updates, tweaks, remade work feat(content, port): UI, accessibility updates, tweaks, remade work May 31, 2024
@Kenan2000
Copy link
Collaborator Author

Closing this because pull request of @KheirFerrum includes fixed version of this work

@Kenan2000 Kenan2000 closed this Jun 1, 2024
@Kenan2000 Kenan2000 deleted the UIaccessibility_portagain_update3 branch June 1, 2024 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data PRs related to datas. Won't crash game (probably) docs PRs releated to docs page src PR changes related to source code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant