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

Rugged Fighter doesn't affect character sheet pain levels #1602

Open
1 task done
l0b0 opened this issue Feb 8, 2023 · 5 comments
Open
1 task done

Rugged Fighter doesn't affect character sheet pain levels #1602

l0b0 opened this issue Feb 8, 2023 · 5 comments
Labels
Bug A bug Domain: Character Sheet Concerns the character sheet Low Priority Matches the goals but is either not relevant for a lot of users or does not add enough value

Comments

@l0b0
Copy link

l0b0 commented Feb 8, 2023

Please confirm the following

Describe the bug

I believe "1/4 lost (Pain +1)" should not show up in the character sheet if I've got the Rugged Fighter advantage.

To Reproduce

  1. Create a character with Rugged Fighter advantage
  2. Go to the character sheet

Expected behavior

"1/2 lost (Pain +1)" should be the first pain level shown in the "Life Points" section

Screenshots

Screenshot from 2023-02-08 20-18-54

Operating System

Linux

Version

1.5.1

Additional context

No response

@l0b0 l0b0 added the Bug A bug label Feb 8, 2023
@l0b0 l0b0 changed the title Rugged Fighter doesn't affect charactter sheet pain levels Rugged Fighter doesn't affect character sheet pain levels Feb 8, 2023
@elyukai
Copy link
Owner

elyukai commented Feb 8, 2023

Thank you so much for your big report(s)!

I did not think that far there, to be honest, it's just a static text I've taken over from other character sheets. But you're totally right.

Since this is not a critical bug, I have to delay its fix a bit, tho, since there are just way too many other issues around.

@elyukai elyukai added Domain: Character Sheet Concerns the character sheet Low Priority Matches the goals but is either not relevant for a lot of users or does not add enough value labels Feb 8, 2023
@l0b0
Copy link
Author

l0b0 commented Feb 8, 2023

Would it be easy to omit the first line conditionally? Even if that means the rest of the lines have a bit of extra spacing above them it would be no big deal.

@elyukai
Copy link
Owner

elyukai commented Feb 9, 2023

I guess that should not be that hard. I used flexbox layout quite "aggressively" on the character sheet, so it should be able to adjust.

I’m currently thinking of splitting the texts to extract the (Pain +1) part, so that I could dynamically add it. Since I’m currently working on a new MAJOR version and I don't expect another V1 release before, I’ll try to include that change in the database rework, so that it can be taken into account for a V2 release.

@ChrisHap
Copy link

ChrisHap commented May 1, 2023

The rules for Rugged Fighter state, that you "Ignore the effects of the highest level of the condition Pain you possess" and that "Pain of Level IV still incapacitates you.".

Also the rule, that you fall unconcious if you are effected by 8 or more conditions, still applies.
So the current implementation should be correct, even when you include Rugged Fighter.

@Wujekklawy
Copy link

you still suffer from Pain I with Rugged Fighter, you just don't get its effects. you still drop unconscious with Pain IV, which is 5hp or in your case 12hp and + Pain from a spell or ability

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug Domain: Character Sheet Concerns the character sheet Low Priority Matches the goals but is either not relevant for a lot of users or does not add enough value
Projects
None yet
Development

No branches or pull requests

4 participants