Skip to content

[HOLD for @0xmiroslav payment] Make Profile > Personal Details > Address true push-to-page fields.  #17548

Closed

Description

@JmillsExpensify

@Beamanator @cristipaval I was in a thread with @shawnborton and he noticed that both our State and Country fields are not true push-to-page inputs. Mind helping us understand why that's the case? Ideally if we're going to allow selection of a list, we'd follow a true push-to-page pattern where the row/input has a right caret, upon which tapping that takes you to the full list on a separate page. Right? Adding a New Feature label on this one given that it's not technically a bug.

Screenshot 2023-04-17 at 17 50 04

In addition to the above, let's make sure we have 20px below each input grouping (could be just an input, or input + hint text), as @shawnborton asked here.

Activity

  1. MelvinBot commented on Apr 17, 2023

    @MelvinBot
    Contributor

    Current assignee @JmillsExpensify is eligible for the NewFeature assigner, not assigning anyone new.

  2. JmillsExpensify commented on Apr 17, 2023

    @JmillsExpensify
    ContributorAuthor

    Went ahead and assigned myself so no one is assigned and we align on next steps.

  3. cristipaval commented on Apr 18, 2023

    @cristipaval
    Contributor

    I think we refactored the parent Profile page but we missed this page.
    It looks like there was an issue created for this but there was confusion here and the issue was closed. cc @Beamanator

  4. JmillsExpensify commented on Apr 18, 2023

    @JmillsExpensify
    ContributorAuthor

    Ok no worries. We can just use this issue to close the loop. Sound good?

  5. cristipaval commented on Apr 18, 2023

    @cristipaval
    Contributor

    Sounds good to me. I can take this one and offload it to Callstack, as it is just front-end work.

  6. Prince-Mendiratta commented on Apr 18, 2023

    @Prince-Mendiratta
    Contributor

    Hi @JmillsExpensify @cristipaval! I'd like to work on this, would it be possible to make this issue external or will you be going ahead with Callstack?

  7. self-assigned this
    on Apr 20, 2023
  8. gedu commented on Apr 20, 2023

    @gedu
    Contributor

    Hey hey, Edu here, I am from Callstack - expert contributor group, I can take a look at this one

  9. cristipaval commented on Apr 20, 2023

    @cristipaval
    Contributor

    Hey @gedu! Thanks for taking this over! Push-to-page design means that the user should be redirected to a new page to select the option, instead of the system picker. Take a look at how pronouns and Timezone selectors work in the Profile page.

  10. gedu commented on Apr 20, 2023

    @gedu
    Contributor

    @cristipaval I'm not seeing that State has a dropdown where is that or how can I reproduce it? (I'm up to date with main)

    Screen Shot 2023-04-20 at 15 55 25

  11. cristipaval commented on Apr 20, 2023

    @cristipaval
    Contributor

    You first have to select the country, which I think isn't intuitive and also is not user-friendly to go down to the latest field, select an option, and then go back up to the State field to select an option. @JmillsExpensify @shawnborton What do you think? Shouldn't we reorder the fields to have Country before State, since the State options depend on the Country selection?

  12. 68 remaining items

  13. melvin-bot commented on Aug 29, 2023

    @melvin-bot

    @JmillsExpensify, @gedu, @mountiny, @0xmiroslav Eep! 4 days overdue now. Issues have feelings too...

  14. JmillsExpensify commented on Aug 30, 2023

    @JmillsExpensify
    ContributorAuthor

    Offer extended to @s77rt. @0xmiroslav I think there's still some uncertainty how you're being paid out. Please let me know how you'd like to proceed.

  15. JmillsExpensify commented on Aug 30, 2023

    @JmillsExpensify
    ContributorAuthor

    @s77rt paid for this job. Going to add a hold for @0xmiroslav.

  16. changed the title [-][HOLD for payment 2023-08-03] Make Profile > Personal Details > Address true push-to-page fields. [/-] [+][HOLD for @0xmiroslav payment] Make Profile > Personal Details > Address true push-to-page fields. [/+] on Aug 30, 2023
  17. added and removed on Sep 4, 2023
  18. mountiny commented on Sep 4, 2023

    @mountiny
    Contributor

    making it monthly for the payment to miroslav

  19. JmillsExpensify commented on Sep 25, 2023

    @JmillsExpensify
    ContributorAuthor

    @0xmiroslav Can we issue payment on this one yet?

  20. 0xmiroslav commented on Sep 26, 2023

    @0xmiroslav
    Contributor

    Not yet

  21. JmillsExpensify commented on Oct 10, 2023

    @JmillsExpensify
    ContributorAuthor

    I'm going to close this because I have an unsustainable amount of GH issue assigned. Please keep track on this one your side and let us know when you are ready to receive payment. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionMonthlyKSv2NewFeatureSomething to build that is a new item.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions