Skip to content

Conversation

@AlexJacksonDS
Copy link
Contributor

My details as card and view notification preferences.
image
image

Notification preferences not as tags
image
image

Copy link
Contributor

@JBsoftwire JBsoftwire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Copy link
Contributor

@stellake stellake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - only minor comments. Could we also add some bottom padding to Subscribed/Unsubscribed text in mobile mobile?

@AlexJacksonDS AlexJacksonDS force-pushed the HEEDLS-544-my-account-user-feedback branch from 929cbbb to f0ee0ba Compare July 13, 2021 11:46
@AlexJacksonDS
Copy link
Contributor Author

Not sure why I needed to force push the branch, something strange happened I guess

New padding:
image

Copy link
Contributor

@stellake stellake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! 👍

@AlexJacksonDS AlexJacksonDS merged commit 4209748 into master Jul 13, 2021
@AlexJacksonDS AlexJacksonDS deleted the HEEDLS-544-my-account-user-feedback branch July 13, 2021 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants