Skip to content

Fixes #39495 - Update Ansible DualList to PF5#796

Open
pondrejk wants to merge 1 commit into
theforeman:masterfrom
pondrejk:pfdual
Open

Fixes #39495 - Update Ansible DualList to PF5#796
pondrejk wants to merge 1 commit into
theforeman:masterfrom
pondrejk:pfdual

Conversation

@pondrejk

@pondrejk pondrejk commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Overview of Changes

before:

Screenshot 2026-07-03 at 10 54 29

after

Screenshot 2026-07-03 at 10 53 05

@pondrejk

pondrejk commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

updated the PR with addition of a tooltip that will make possible reading longer item names on narrow screen or modal

Comment thread webpack/components/DualList/index.js Outdated
onSetOrderedItems={onChosenSetOrder}
/>
</div>
status={`${selectState.availableSelected.length} of ${props.availableOptions.length} options selected`}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should be translated

Comment thread webpack/components/DualList/index.js Outdated
>
<DualListSelectorPane
title={__('Assigned Ansible roles')}
status={`${selectState.chosenSelected.length} of ${props.chosenOptions.length} options selected`}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should be translated

@Lukshio

Lukshio commented Jul 22, 2026

Copy link
Copy Markdown

@pondrejk theforeman/foreman#11018 has been merged, so you can reuse the component

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.

2 participants