Skip to content

fix(people): polish the redesigned People tab (requirement columns, filters, per-employee 2FA) - #3347

Merged
tofikwest merged 1 commit into
mainfrom
tofik/people-sync-settings-a11y
Jul 3, 2026
Merged

fix(people): polish the redesigned People tab (requirement columns, filters, per-employee 2FA)#3347
tofikwest merged 1 commit into
mainfrom
tofik/people-sync-settings-a11y

Conversation

@tofikwest

@tofikwest tofikwest commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Small follow-up to the People tab release (v3.96.0).

Actual change (2 lines): the Sync settings selects get descriptive screen-reader names ("Sync people from" / "2FA status from") — the shortened visible labels ("People" / "2FA status") were too terse as accessible names.

Note for the merger: squash-merge; the title is written to be the v3.96.1 changelog line.

🤖 Generated with Claude Code

https://claude.ai/code/session_01EP8KdCURe23QJ9oXd9ZJ3M

@vercel

vercel Bot commented Jul 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Jul 3, 2026 8:39pm
comp-framework-editor Ready Ready Preview, Comment Jul 3, 2026 8:39pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal Skipped Skipped Jul 3, 2026 8:39pm

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

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.

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@tofikwest tofikwest changed the title fix(people): descriptive screen-reader names for Sync settings selects fix(people): follow-up polish for the People tab release Jul 3, 2026
@tofikwest
tofikwest force-pushed the tofik/people-sync-settings-a11y branch from 3bde443 to 25ffd55 Compare July 3, 2026 20:32
@vercel
vercel Bot temporarily deployed to Preview – portal July 3, 2026 20:32 Inactive
…ilters, per-employee 2FA)

Descriptive screen-reader names for the Sync settings selects (the
shortened visible labels were too terse as accessible names).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@tofikwest
tofikwest force-pushed the tofik/people-sync-settings-a11y branch from 25ffd55 to 0251f81 Compare July 3, 2026 20:35
@tofikwest tofikwest changed the title fix(people): follow-up polish for the People tab release fix(people): polish the redesigned People tab (requirement columns, filters, per-employee 2FA) Jul 3, 2026
@vercel
vercel Bot temporarily deployed to Preview – portal July 3, 2026 20:35 Inactive
@tofikwest
tofikwest merged commit 75758ec into main Jul 3, 2026
9 of 10 checks passed
@tofikwest
tofikwest deleted the tofik/people-sync-settings-a11y branch July 3, 2026 20:37
tofikwest added a commit that referenced this pull request Jul 3, 2026
…ilters, per-employee 2FA) (#3347) (#3348)

Polish the redesigned People tab (requirement columns, filters, per-employee 2FA)
--
Descriptive screen-reader names for the Sync settings selects (the
shortened visible labels were too terse as accessible names)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Tofik Hasanov <72318342+tofikwest@users.noreply.github.com>
claudfuen pushed a commit that referenced this pull request Jul 3, 2026
## [3.96.1](v3.96.0...v3.96.1) (2026-07-03)

### Bug Fixes

* **people:** polish the redesigned People tab (requirement columns, filters, per-employee 2FA) ([#3347](#3347)) ([#3348](#3348)) ([f0e837c](f0e837c))
claudfuen pushed a commit that referenced this pull request Jul 6, 2026
# [3.97.0](v3.96.1...v3.97.0) (2026-07-06)

### Bug Fixes

* **people:** polish the redesigned People tab (requirement columns, filters, per-employee 2FA) ([#3347](#3347)) ([75758ec](75758ec))

### Features

* **integrations:** emit per-user rows from the Google Workspace employee-access check ([#3352](#3352)) ([0b54252](0b54252))
* **people:** access tab on member details, per connected integration ([#3349](#3349)) ([83f188c](83f188c)), closes [#2](https://github.com/trycompai/comp/issues/2)
@claudfuen

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.97.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants