Skip to content

docs(dual list selector): v6 design guidelines image updates#4598

Merged
andrew-ronaldson merged 8 commits into
patternfly:mainfrom
kaylachumley:iss4523
May 8, 2025
Merged

docs(dual list selector): v6 design guidelines image updates#4598
andrew-ronaldson merged 8 commits into
patternfly:mainfrom
kaylachumley:iss4523

Conversation

@kaylachumley
Copy link
Copy Markdown
Collaborator

close #4523

@kaylachumley kaylachumley changed the title docs(dual-list-selector): v6 design guidelines updates docs(dual list selector): v6 design guidelines image updates May 5, 2025
@patternfly-build
Copy link
Copy Markdown
Collaborator

patternfly-build commented May 5, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In other guidelines where there's a sequence of events, the number on the screenshot matches the sequence number, and they're shown in that order. Because there are several callouts for the On Drag example, though, I think the numbered annotations are a bit confusing, especially because there are multiple #1s. Maybe @edonehoo can weigh in on the best way to handle this?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I agree! i think the numbered annotations need removed from this section

Copy link
Copy Markdown
Collaborator

@edonehoo edonehoo May 6, 2025

Choose a reason for hiding this comment

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

or I think if you swap the order so that the pictures come before the numbered descriptions in the content it would be the most helpful potentially

Like

[first image in the sequence]

  1. onDrag event ...

[second image] -- remove the "1" annotation
2. Ghost row...

[third image] -- change the annotation # in this image to be "3"
3. postDrag event ...

Does this make sense?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Maybe leaving the images after the text would be fine too as long as you change the annotations in the second and third images

Copy link
Copy Markdown
Contributor

@bekah-stephens bekah-stephens left a comment

Choose a reason for hiding this comment

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

I left a comment on the On Drag and Post Drag section - just some food for thought

@kaylachumley
Copy link
Copy Markdown
Collaborator Author

pushed the newest updates based on your comments above! @edonehoo @bekah-stephens

Copy link
Copy Markdown
Contributor

@bekah-stephens bekah-stephens left a comment

Choose a reason for hiding this comment

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

looks good!

@andrew-ronaldson andrew-ronaldson merged commit abd4ebd into patternfly:main May 8, 2025
4 checks passed
@kaylachumley kaylachumley deleted the iss4523 branch May 21, 2025 14:39
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.

Guideline updates: Dual list selector

5 participants