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

[#4619] Increase "Add X" button visibility in activity sheets #4971

Open
wants to merge 5 commits into
base: 4.2.x
Choose a base branch
from

Conversation

roth-michael
Copy link
Contributor

@roth-michael roth-michael commented Jan 7, 2025

Would close #4619

Open to suggestions re: styling (definitely needs some improvement, I think - e.g. "Create Effect" not being centered; but I didn't center it because I kind of think it looks better that way) and I reused the tooltip i18n but considered creating an Add-type localization instead.

Consumption tab:
Screenshot 2025-01-06 195438

Attack "Effect" tab:
Screenshot 2025-01-06 195603

Damage "Effect" tab:
Screenshot 2025-01-06 195755

Enchanting tab:
Screenshot 2025-01-06 195822

Save "Effect" tab:
Screenshot 2025-01-06 195843

Summoning tab:
Screenshot 2025-01-06 195939

@thatlonelybugbear
Copy link
Contributor

How about alinging it to the right, or even reusing the same system's Create New XXX + buttons, which are already present in different places? 🤔

@roth-michael
Copy link
Contributor Author

image
Not sure I'm a fan of right-aligned. But dunno, really. As for the localization, what such buttons are you thinking of? Outside of activities, it's almost all handled via the little gold "create child" square buttons at the bottom right of the sheet, which are unlabeled.

@thatlonelybugbear
Copy link
Contributor

Yeah I meant exactly those ones without a label (well maybe a hint on hover).

It already has made it in the UI in different places and probably would make sense to users without the need of the actual text, which I am not a fan of TBH 🤷

@roth-michael
Copy link
Contributor Author

I see - so like this:
image

Not entirely sold on that, lotta blank space.

@roth-michael
Copy link
Contributor Author

Aligning to the left and giving 2px top margin:
image
image
That's feeling better to me.

@arbron
Copy link
Collaborator

arbron commented Jan 7, 2025

Centered version is basically what I used on Black Flag:

Screenshot 2025-01-07 at 11 51 08

@arbron arbron added ui User interface related features or bugs system: activities labels Jan 7, 2025
@roth-michael
Copy link
Contributor Author

I think part of the issue with just centered is that without the dotted line you've got in that screenshot (which might not fit in with 5e's styling), when it follows text like the "allow critical" hint it looks a bit strange. Aligning to the left makes it look a lot more natural in that instance, I think.

Copy link
Collaborator

@arbron arbron left a comment

Choose a reason for hiding this comment

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

Make sure you update the "Create Recovery Profile" button on item sheets to match.

The color for the button in light mode seems a bit too light, feels like it is disabled, but I'll leave any color decision to Kim.

templates/activity/parts/activity-consumption.hbs Outdated Show resolved Hide resolved
templates/activity/parts/activity-effects.hbs Outdated Show resolved Hide resolved
templates/activity/parts/attack-damage.hbs Outdated Show resolved Hide resolved
templates/activity/parts/damage-damage.hbs Outdated Show resolved Hide resolved
templates/activity/parts/enchant-enchantments.hbs Outdated Show resolved Hide resolved
templates/activity/parts/save-damage.hbs Outdated Show resolved Hide resolved
templates/activity/parts/summon-profiles.hbs Outdated Show resolved Hide resolved
templates/shared/fields/field-uses.hbs Outdated Show resolved Hide resolved
@arbron arbron added this to the D&D5E 4.2.0 milestone Jan 16, 2025
@arbron arbron requested a review from Fyorl January 16, 2025 18:29
@arbron arbron modified the milestones: D&D5E 4.2.0, D&D5E 4.3.0 Jan 20, 2025
@arbron
Copy link
Collaborator

arbron commented Jan 20, 2025

@roth-michael Give the original issue a comment so I can assign it to you :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
system: activities ui User interface related features or bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider improving contrast for + button in field legend
3 participants