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

CFG: controller binds, doom style weapon switch and config changes #234

Merged
merged 2 commits into from
Nov 9, 2024

Conversation

lavenderdotpet
Copy link
Owner

@lavenderdotpet lavenderdotpet commented Nov 9, 2024

Description of Changes


improved controller support + doom style weapon switch as an optional toggle in the config keyboard and a default for controller on the dpad

also turned cl_alwaysrun to 1 which is "quakespasm" which means u press shift to slow walk

Visual Sample


video.mp4

Checklist


  • I have read the LibreQuake contribution guidelines
  • I have thoroughly tested my changes to the best of my ability
  • I confirm I have not contributed anything that would impact LibreQuake's licensing and usage
  • This Pull Request fixes a critical issue that should be reviewed and merged as soon as possible

marked as critical for getting the next release out today

improved controller support + doom style weapon switch as an optional toggle in the config keyboard and a default for controller on the dpad
lq1/default.cfg Show resolved Hide resolved
lq1/default.cfg Show resolved Hide resolved
Copy link
Collaborator

@RandomBrushes RandomBrushes left a comment

Choose a reason for hiding this comment

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

Very clever weapon switch mechanic. Looks good to me.

Copy link
Collaborator

@RandomBrushes RandomBrushes left a comment

Choose a reason for hiding this comment

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

This time, actually approve (instead of comment)

@lavenderdotpet lavenderdotpet merged commit eef5d0b into main Nov 9, 2024
12 checks passed
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.

4 participants