Skip to content

Commit 0ad3a8c

Browse files
authored
fix: remove padding and border radius from order type bottom sheet (MetaMask#22143)
## **Description** Remove padding and border radius from order type bottom sheet ## **Changelog** CHANGELOG entry:null ## **Related issues** Fixes: https://consensyssoftware.atlassian.net/browse/MDP-392 ## **Manual testing steps** ```gherkin Feature: my feature name Scenario: user [verb for user action] Given [describe expected initial app state] When user [verb for user action] Then [describe expected outcome] ``` ## **Screenshots/Recordings** `~` ### **Before** <img width="500" alt="image" src="https://github.com/user-attachments/assets/2662e9b0-adaa-4fc3-b89d-128b32f2c478" /> ### **After** <img width="500" alt="Simulator Screenshot - iPhone 16 Pro Max - 2025-11-04 at 11 22 56" src="https://github.com/user-attachments/assets/84b8a77e-5512-4a6c-9876-13c9a4f8e7ae" /> ## **Pre-merge author checklist** - [x] 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 - [x] 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. ## **Pre-merge reviewer checklist** - [x] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed). - [x] 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] > Removes container horizontal padding and option border radius in `PerpsOrderTypeBottomSheet` styles. > > - **UI/Styles** (`app/components/UI/Perps/components/PerpsOrderTypeBottomSheet/PerpsOrderTypeBottomSheet.styles.ts`): > - Remove `container` `paddingHorizontal`. > - Remove `option` `borderRadius`. > - Keep spacing and selection colors unchanged. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 99111f7. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
1 parent f4d7cba commit 0ad3a8c

1 file changed

Lines changed: 0 additions & 2 deletions

File tree

app/components/UI/Perps/components/PerpsOrderTypeBottomSheet/PerpsOrderTypeBottomSheet.styles.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,11 @@ import { Theme } from '../../../../../util/theme/models';
44
export const createStyles = (colors: Theme['colors']) =>
55
StyleSheet.create({
66
container: {
7-
paddingHorizontal: 16,
87
paddingVertical: 24,
98
},
109
option: {
1110
paddingVertical: 16,
1211
paddingHorizontal: 16,
13-
borderRadius: 12,
1412
marginBottom: 16,
1513
},
1614
optionSelected: {

0 commit comments

Comments
 (0)