-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[waiting for payment][$250] Share code- QR code expands to full size after opening for WS rooms, tasks and threads #44436
Comments
Triggered auto assignment to @adelekennedy ( |
Triggered auto assignment to @danieldoglas ( |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
@adelekennedy I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors. |
We think this issue might be related to the #vip-vsb |
ProposalPlease re-state the problem that we are trying to solve in this issue.QRCode change size after layout change What is the root cause of that problem?Default QR code size is 120px, after layout change the QR code recalculate size at this line App/src/components/QRShare/index.tsx Line 32 in 224f09e
so the UI QRCode will update width and height What changes do you think we should make in order to solve the problem?We have to set the default initial height for qrcode size at this line base on platform App/src/components/QRShare/index.tsx Line 19 in 224f09e
ST like that + const isMobilePlatform = Platform.OS === 'android' || Platform.OS === 'ios';
+ const isMobileBrowser = Browser.isMobile();
+ const isMobile = isMobilePlatform || isMobileBrowser;
+ const { windowWidth } = useWindowDimensions();
+. const qrShareContainerWidth = isMobile ? windowWidth : variables.sideBarWidth;
- const [qrCodeSize, setQrCodeSize] = useState<number | undefined>();
+ // styles.ph5.paddingHorizontal * 2: This is the padding outside of the QR code.
+ // variables.qrShareHorizontalPadding * 2: This is the padding inside of the QR code.
+ // default size is remaining width after minus padding from parent container
+ const [qrCodeSize, setQrCodeSize] = useState<number>(
+ qrShareContainerWidth - (styles.ph5.paddingHorizontal * 2) - (variables.qrShareHorizontalPadding * 2)
+ );
POC
Screen_Recording_20240706_090235_New.Expensify.Dev.mp4What alternative solutions did you explore? (Optional) |
ProposalPlease re-state the problem that we are trying to solve in this issue.
What is the root cause of that problem?
What changes do you think we should make in order to solve the problem?
What alternative solutions did you explore? (Optional) |
Job added to Upwork: https://www.upwork.com/jobs/~01e0a4dbc404a8ab34 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @alitoshmatov ( |
@danieldoglas, @alitoshmatov, @adelekennedy Huh... This is 4 days overdue. Who can take care of this? |
@dominictb I understand your concern, and I am sorry that this time, it did not work out. I think both of you and your solutions will work, and they are similar, so @huult should go ahead, as their proposal came in first. @huult what is your ETA for the PR? |
I will create the pull requests soon. The ETA is the next day. Thank you. |
@mountiny @danieldoglas Thanks for your answer |
here is my PR |
Hi @mountiny , @danieldoglas , My PR was merged to production a week ago, but I noticed the Mevin bot isn't working as expected compared to other tickets I've worked on. Can I ask if this ticket is currently able to process a payment? |
ty @danieldoglas let me handle payment now: Payouts due:
Upwork job is here. |
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed @alitoshmatov
|
@alitoshmatov checklist above when you get a chance! |
Regression Test Proposal
Do we agree 👍 or 👎 cc: @adelekennedy |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 9.0.2-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4675372
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Expected Result:
The QR code is displayed fully
Actual Result:
The QR code is displayed small briefly and expands to the full size
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Bug6524801_1719378147729.video_2024-06-26_00-59-25.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @adelekennedyThe text was updated successfully, but these errors were encountered: