-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[$250] Misaligned Arrow Icons in Trip Summary #61740
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
Comments
Triggered auto assignment to @bfitzexpensify ( |
This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989 |
ProposalPlease re-state the problem that we are trying to solve in this issue.Misaligned Arrow Icons in Trip Summary What is the root cause of that problem?Arrow icon not centered. What changes do you think we should make in order to solve the problem?Add additional styles here to horizontally center the arrow. additionalStyles={styles.alignSelfCenter}
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?NA What alternative solutions did you explore? (Optional)Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job. |
|
cc @Expensify/design - kinda looks like we have set this to always be a certain distance away from the destination station, but perhaps we should have it equidistant between the departure and destination stations? Not 100% sure where we set this - perhaps in https://github.com/Expensify/Expensify/issues/390709? |
Yeah I kinda feel like each location (start and stop) should have equal width and the arrow should sit perfectly in between them. That way the arrows should always line up down the center. Does that make sense? |
Job added to Upwork: https://www.upwork.com/jobs/~021920839738421141603 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @Ollyws ( |
That makes sense to me, and then maybe we truncate after 2 lines for both the left and right sides of the arrow? I think we'll eventually revisit this design entirely but I think that's a good path forward for now. |
Please re-state the problem that we are trying to solve in this issue. The arrow icon between the origin and destination station names in the Trip Summary is not vertically aligned with the text. This misalignment creates a visually unbalanced look across platforms, particularly noticeable on mobile and web. What is the root cause of that problem? What changes do you think we should make in order to solve the problem? What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?
What alternative solutions did you explore? (Optional) I considered using inline text formatting or inserting a Unicode arrow directly within the text. However, this approach didn’t offer precise control over alignment and spacing, and results varied across platforms. A more flexible layout approach is the better solution. |
📣 @khhamdan! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
ProposalPlease re-state the problem that we are trying to solve in this issue.Misaligned Arrow Icons in Trip Summary. What is the root cause of that problem?Currently we use
This causes the text to spread, this is the RCA, we don't have any way to align the items propperly What changes do you think we should make in order to solve the problem?According to the comments from design team here and here, we should update the styling a bit:
titleComponent = (
<View style={[styles.flexRow, styles.alignItemsCenter]}>
<View style={[styles.flex1, styles.flexRow]}>
<Text
style={[styles.labelStrong]}
numberOfLines={2}
>
{startName}
</Text>
</View>
<View
style={{
width: variables.iconSizeSmall + 8, // icon width + some margin (adjust as needed)
alignItems: 'center',
justifyContent: 'center',
}}
>
<Icon
src={Expensicons.ArrowRightLong}
width={variables.iconSizeSmall}
height={variables.iconSizeSmall}
fill={theme.icon}
/>
</View>
<View style={[styles.flex1, styles.flexRow, styles.justifyContentEnd]}>
<Text
style={[styles.labelStrong, styles.textAlignRight]}
numberOfLines={2}
>
{endName}
</Text>
</View>
</View>
); we can render the previous code for flight details and this current code for train details specifically, if we want both of them to be separate. minor design changes can be done during PR phase What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?N/A ui bug What alternative solutions did you explore? (Optional)N/A |
is this issue resolved? |
📣 @RitvikSh! 📣
|
@twilight2294's proposal LGTM. |
Triggered auto assignment to @rafecolton, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
I like that idea 👍 |
📣 @twilight2294 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
Uh oh!
There was an error while loading. Please reload this page.
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number:
Reproducible in staging?: Needs Reproduction
Reproducible in production?: Needs Reproduction
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: @coleaeason
Slack conversation (hyperlinked to channel name): #Travel
Action Performed:
Expected Result:
Arrows should be vertically and horizontally aligned
Actual Result:
Arrows appear off-center or misaligned
Workaround:
Unknown
Platforms:
Select the officially supported platforms where the issue was reproduced:
Platforms Tested:
On which of our officially supported platforms was this issue tested:Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @twilight2294The text was updated successfully, but these errors were encountered: