Skip to content

Commit 5b1913f

Browse files
authored
chore: remove unused SegmentedControl temp component (#29846)
## **Description** Removes the unused temporary SegmentedControl component from `app/component-library/components-temp/SegmentedControl`. This also removes the stale Storybook registration entry for its story. A reference search confirms there are no remaining app, test, or Storybook references. ## **Changelog** CHANGELOG entry: null ## **Related issues** Fixes: ## **Manual testing steps** ```gherkin Feature: temporary component cleanup Scenario: SegmentedControl temp component is removed Given the repository is on this branch When searching for SegmentedControl temp component references Then no app, test, or Storybook references are found ``` ## **Screenshots/Recordings** N/A. Code cleanup only. ### **Before** N/A ### **After** N/A ## **Validation** - `rg -n "components-temp/SegmentedControl|SegmentedControl" app tests .storybook --glob '!node_modules/**'` returned no matches. ## **Pre-merge author checklist** - [ ] I've followed [MetaMask Contributor Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile Coding Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md). - [x] I've completed the PR template to the best of my ability - [x] I've included tests if applicable - [x] I've documented my code using [JSDoc](https://jsdoc.app/) format if applicable - [ ] I've applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors. #### Performance checks (if applicable) - [ ] I've tested on Android - [ ] I've tested with a power user scenario - [ ] I've instrumented key operations with Sentry traces for production performance metrics ## **Pre-merge reviewer checklist** - [ ] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed). - [ ] I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots. <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk cleanup that only deletes an unused temp UI component and its Storybook wiring; primary risk is accidental removal if any downstream consumer existed outside the searched paths. > > **Overview** > Removes the temporary `components-temp/SegmentedControl` component entirely (implementation, types/constants, styles, tests, README, and Storybook story). > > Updates Storybook’s auto-generated `storybook.requires.js` to drop the stale import/registration for the deleted `SegmentedControl` story. > > <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit 2b847b4. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup> <!-- /CURSOR_SUMMARY -->
1 parent 58ca663 commit 5b1913f

9 files changed

Lines changed: 0 additions & 1226 deletions

File tree

.storybook/storybook.requires.js

Lines changed: 0 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

app/component-library/components-temp/SegmentedControl/README.md

Lines changed: 0 additions & 196 deletions
This file was deleted.

app/component-library/components-temp/SegmentedControl/SegmentedControl.constants.ts

Lines changed: 0 additions & 45 deletions
This file was deleted.

0 commit comments

Comments
 (0)