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

RHCLOUD-32197 Support Case Widget updates #598

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

aferd
Copy link
Contributor

@aferd aferd commented Jun 20, 2024

Description

Add filtering and sorting to support case widget table.

RHCLOUD-32197


Screenshots

Before:

Screenshot 2024-06-20 at 2 55 29 PM

After:

Screenshot 2024-06-20 at 2 54 31 PM


Checklist ☑️

  • PR only fixes one issue or story

  • Change reviewed for extraneous code

  • UI best practices adhered to

  • Commits squashed and meaningfully named

  • All PR checks pass locally (build, lint, test, E2E)

  • (Optional) QE: Needs QE attention (OUIA changed, perceived impact to tests, no test coverage)
  • (Optional) QE: Has been mentioned
  • (Optional) UX: Needs UX attention (end user UX modified, missing designs)
  • (Optional) UX: Has been mentioned

@aferd aferd marked this pull request as draft June 20, 2024 19:09
Comment on lines +155 to +190
const severityMenuItems = (
<SelectList>
<SelectOption
hasCheckbox
key={severityTypes.low}
value={severityTypes.low}
isSelected={filters.severity.includes(severityTypes.low)}
>
{severityTypes.low}
</SelectOption>
<SelectOption
hasCheckbox
key={severityTypes.normal}
value={severityTypes.normal}
isSelected={filters.severity.includes(severityTypes.normal)}
>
{severityTypes.normal}
</SelectOption>
<SelectOption
hasCheckbox
key={severityTypes.high}
value={severityTypes.high}
isSelected={filters.severity.includes(severityTypes.high)}
>
{severityTypes.high}
</SelectOption>
<SelectOption
hasCheckbox
key={severityTypes.urgent}
value={severityTypes.urgent}
isSelected={filters.severity.includes(severityTypes.urgent)}
>
{severityTypes.urgent}
</SelectOption>
</SelectList>
);
Copy link
Contributor

Choose a reason for hiding this comment

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

This could be definitely deduplicated - generated just from array of severity variants

setIsStatusExpanded(!isStatusExpanded);
};

const statusMenuItems = (
Copy link
Contributor

Choose a reason for hiding this comment

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

this could be deduplicated as well

</SelectList>
);

const toggleGroupItems = (
Copy link
Contributor

Choose a reason for hiding this comment

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

this could be auto-generated as well

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.

None yet

2 participants