chore: make the entire row clickable in logs quick filters

[vikrantgupta25]

Summary

• currently only the chevron icon was clickable to expand and collpase the quick filters in logs.
• made the entire row clickable for expand and collapse for better experience

Related Issues / PR's

contributes to - #6572

Screenshots

Screen.Recording.2024-12-03.at.12.08.22.AM.mov

Affected Areas and Manually Tested Areas

• quick filters in logs

---

Important

Make entire row clickable for expand/collapse in logs quick filters, enhancing user experience.

• Behavior:
  • Entire row in filter-header-checkbox is now clickable for expand/collapse in Checkbox.tsx.
  • Clear All button click event now stops propagation to prevent row toggle.
• Styles:
  • Added cursor: pointer to .filter-header-checkbox in Checkbox.styles.scss to indicate clickable area.

^{This description was created by for 2de8dac. It will automatically update as commits are pushed.}

[github-actions]

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 2de8dac in 25 seconds

More details

• Looked at 67 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 3 drafted comments based on config settings.

1. frontend/src/components/QuickFilters/FilterRenderers/Checkbox/Checkbox.tsx:412

• Draft comment:
  The cursor="pointer" on the chevron icon is redundant since the entire row is now clickable. Consider removing it for clarity.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR aims to make the entire row clickable for expanding and collapsing the quick filters. The change involves moving the click handler from the chevron icon to the entire row. However, the chevron icon still has a cursor style applied, which is now redundant since the entire row is clickable.

2. frontend/src/components/QuickFilters/FilterRenderers/Checkbox/Checkbox.tsx:414

• Draft comment:
  The cursor="pointer" on the chevron icon is redundant since the entire row is now clickable. Consider removing it for clarity. This applies to both ChevronDown and ChevronRight icons.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR introduces a click handler on the entire row, but the chevron icons still have a cursor style applied, which is now redundant. This is also applicable to the ChevronRight icon.

3. frontend/src/components/QuickFilters/FilterRenderers/Checkbox/Checkbox.tsx:412

• Draft comment:
  Avoid using inline styles for the cursor property. Use CSS classes or styled components instead. This applies to line 414 as well.
• Reason this comment was not posted:
  Comment was on unchanged code.

Workflow ID: wflow_xvSRR3YZNfM7jVy2

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.