fix(ui): remove table-fixed layout causing column cramping on Firefox

[jamcalli]

Description

Related Issues

Type of Change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Performance improvement
• Code refactoring
• Documentation update
• Dependency update

Testing Performed

Screenshots

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• My changes work with existing functionality

Summary by CodeRabbit

• Style
  • Improved table layout flexibility to better accommodate dynamic content
  • Refined background styling for enhanced visual distinction between standard, alternating, and selected table rows

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 451504f0-131a-4c70-b901-9accd87cb2eb

📥 Commits

Reviewing files that changed from the base of the PR and between 522d5ad and 2928ec8.

📒 Files selected for processing (1)

• src/client/components/ui/table.tsx

---

Walkthrough

Updated Tailwind CSS classes in the table component: removed the table-fixed layout constraint, and adjusted background opacity values for odd rows (from bg-black/[0.04] to bg-black/4) and selected rows (from bg-black/[0.03] to bg-black/3).

Changes

Cohort / File(s)	Summary
Table Component Styling src/client/components/ui/table.tsx	Removed table-fixed class from table root element; simplified background opacity values for odd rows and selected rows by changing notation from bracket format to shorthand (e.g., bg-black/[0.04] to bg-black/4).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

• [BUG] Firefox mobile regression #1123: Removal of table-fixed class may be related to the Firefox mobile table-layout regression issue mentioned.

Possibly related PRs

• fix(ui): use table-fixed layout to prevent column blowout in Firefox #1114: Directly related as it also modifies Tailwind layout classes in the same table component.

Suggested labels

fix

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: removing the table-fixed layout constraint that was causing column cramping issues on Firefox.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch develop

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}