You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[$250] Incorrect Button Size on "Invite Whisper" Modal #61905
Version Number: 9.1.44-6 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs:https://stackoverflow.com/c/expensify/questions/4856 Expensify/Expensify Issue URL: Issue reported by:@dubielzyk-expensify Slack conversation (hyperlinked to channel name): #Expensify Bugs
Action Performed:
Navigate to any workspace
@mention a user who's not on that workspace
See the message with small buttons
Expected Result:
Buttons should be size "medium"
Actual Result:
Buttons are currently size "small"
Workaround:
Unknown
Platforms:
Select the officially supported platforms where the issue was reproduced:
Android: App
Android: mWeb Chrome
iOS: App
iOS: mWeb Safari
iOS: mWeb Chrome
Windows: Chrome
MacOS: Chrome / Safari
MacOS: Desktop
Platforms Tested:
On which of our officially supported platforms was this issue tested:
Inside InviteToWhisperModal.js, or where the modal is rendered:
<ConfirmModal
...
confirmText "Invite"
onConfirm={handleInvite}
confirmButtonStyles={[styles.buttonLarge, styles.p5, styles.mt5]}
confirmTextStyle={[styles.textStrong, styles.textNormal]}
...
/>
If not using ConfirmModal, and the button is direct:
<Button
text="Invite"
onPress={handleInvite}
style={[styles.buttonLarge, styles.mt5]}
textStyles={[styles.textStrong, styles.textNormal]}
success
/>
This aligns the button size with the design standards used across the app.
📣 @samoousama! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:
Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>
Activity
melvin-bot commentedon May 13, 2025
Triggered auto assignment to @twisterdotcom (
Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.m-natarajan commentedon May 13, 2025
Proposal by @dukenv0307
Proposal
Please re-state the problem that we are trying to solve in this issue.
The buttons on the "invite whisper" are size small
What is the root cause of that problem?
We forgot to add
isMediumSizedastrueinactionableMentionWhisperOptionsaction like we did with othersApp/src/pages/home/report/PureReportActionItem.tsx
Lines 751 to 764 in 12e6cb3
What changes do you think we should make in order to solve the problem?
We should add
isMediumSizedastrueApp/src/pages/home/report/PureReportActionItem.tsx
Lines 751 to 764 in 12e6cb3
Note: We should also check other actionable buttons to see if they are the correct size.
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?
None UI bug
What alternative solutions did you explore? (Optional)
[-]Incorrect Button Size on "Invite Whisper" Modal[/-][+][$250] Incorrect Button Size on "Invite Whisper" Modal[/+]melvin-bot commentedon May 16, 2025
Job added to Upwork: https://www.upwork.com/jobs/~021923391355281009698
melvin-bot commentedon May 16, 2025
Triggered auto assignment to Contributor-plus team member for initial proposal review - @akinwale (
External)samoousama commentedon May 16, 2025
Inside InviteToWhisperModal.js, or where the modal is rendered:
<ConfirmModal
...
confirmText "Invite"
onConfirm={handleInvite}
confirmButtonStyles={[styles.buttonLarge, styles.p5, styles.mt5]}
confirmTextStyle={[styles.textStrong, styles.textNormal]}
...
/>
If not using ConfirmModal, and the button is direct:
<Button
text="Invite"
onPress={handleInvite}
style={[styles.buttonLarge, styles.mt5]}
textStyles={[styles.textStrong, styles.textNormal]}
success
/>
This aligns the button size with the design standards used across the app.
melvin-bot commentedon May 16, 2025
📣 @samoousama! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:
Format:
1 remaining item
twisterdotcom commentedon May 20, 2025
How is this going @akinwale? Any good proposals yet?
akinwale commentedon May 21, 2025
@dukenv0307's proposal works here.
@dukenv0307 could you post a comment on this issue so that you can be assigned?
🎀👀🎀 C+ reviewed.
melvin-bot commentedon May 21, 2025
Triggered auto assignment to @pecanoro, see https://stackoverflow.com/c/expensify/questions/7972 for more details.
pecanoro commentedon May 21, 2025
Assigning @dukenv0307 to the issue
dukenv0307 commentedon May 22, 2025
This issue was fixed in: #62085. So I think we don't need to open the PR cc @akinwale
akinwale commentedon May 22, 2025
In that case, I believe this can be closed with 50% compensation due.
cc @twisterdotcom
twisterdotcom commentedon May 23, 2025
Payment Summary:
JmillsExpensify commentedon Jun 9, 2025
$125 approved for @dukenv0307
JmillsExpensify commentedon Jun 9, 2025
$125 approved for @akinwale