Skip to content
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.

Sign up for GitHub

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

Jump to bottom

BOT PROFILE: Header #1404

Closed
3 tasks done
zavreb opened this issue Oct 16, 2017 · 12 comments
Closed
3 tasks done

BOT PROFILE: Header #1404

zavreb opened this issue Oct 16, 2017 · 12 comments
Assignees
@southerneer
Milestone
Sprint #17.1 | Oc...

Comments

@zavreb
Copy link

zavreb commented Oct 16, 2017 •
edited by southerneer
Loading

GOAL: Update header per Bot Profile redesign.

REQs:

  • Header is static, keep as is
  • Please reenable "Sharing" capability on the top right hand corner, replace Share text with the Share icon, please maintain existing functionality, see here for zeplin link
  • Keep the "<" icon grey
  • See here for zeplin zpl.io/bWJLdPa

Tweaked Header

image
@zavreb zavreb added this to the Sprint #17.1 | Oct 13 - Oct 26 milestone Oct 16, 2017
@aksonov
Copy link
Contributor

aksonov commented Oct 17, 2017

What about Bot header for Bot Map UI?

@zavreb
Copy link
Author

zavreb commented Oct 17, 2017

@aksonov see requirements for ticket #1412

@aksonov aksonov assigned aksonov and southerneer and unassigned aksonov Oct 20, 2017
@southerneer
Copy link
Contributor

@irfirl @zavreb Are we changing the back button to pink for all back buttons in the app?

@irfirl
Copy link
Contributor

irfirl commented Oct 23, 2017

@southerneer we should keep it gray for now

southerneer pushed a commit that referenced this issue Oct 23, 2017
@prefontaineman1
for #1404
5579267
@southerneer southerneer mentioned this issue Oct 23, 2017
Merged
@zavreb
Copy link
Author

zavreb commented Oct 23, 2017 •
edited
Loading

@southerneer updated req. to keep the < grey...

southerneer pushed a commit that referenced this issue Oct 24, 2017
@prefontaineman1
for #1404
8ebb898
@southerneer
Copy link
Contributor

southerneer commented Oct 24, 2017 •
edited
Loading

@zavreb cool. This one's pretty straightforward, but for clarity's sake consider the strikethrough for changing requirements. In this case...

  • [ ] change the "<" icon to pink
  • Keep the "<" icon grey

@mstidham
Copy link

Verified on Staging

1 similar comment
@irfirl
Copy link
Contributor

irfirl commented Oct 27, 2017

Verified on Staging

@zavreb
Copy link
Author

zavreb commented Oct 27, 2017

Verified on Staging. Pushing ticket to "Verify on Prod"

@mstidham
Copy link

mstidham commented Nov 1, 2017

Verified on Prod

1 similar comment
@irfirl
Copy link
Contributor

irfirl commented Nov 1, 2017

Verified on Prod

@zavreb
Copy link
Author

zavreb commented Nov 1, 2017 •
edited
Loading

Verified on Prod. Closing ticket. Though the < is now gray...

@zavreb zavreb closed this as completed Nov 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@southerneer southerneer

Labels
None yet
Projects
None yet
Milestone
Sprint #17.1 | Oct 13 - Oct 26
Development

No branches or pull requests
5 participants
@aksonov @southerneer @irfirl @zavreb @mstidham