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.

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

Change the order of template modes in admin interface #1264

Closed
amedina opened this Issue Jul 10, 2018 · 2 comments

Comments

5 participants
@amedina
Copy link
Member

commented Jul 10, 2018

As a user, I should see "Native" as the first toggleable option for AMPhtml so that I am most likely to choose this option.

  • AC1: Instead of the ordering list of "Classic, Paired, Native" I should see "Native, Paired, Classic" as the three options.

Currently the operational mode of the plugin can be configured in the admin via toggle buttons. The current ordering of the buttons reflects the chronological evolution of the modes: Classic, Paired, Native. Proposal: Change the order to have Native first to implicitly encourage that as the best option.

screen shot 2018-07-10 at 2 24 23 pm

@westonruter westonruter added this to the v1.0 milestone Jul 11, 2018

@westonruter westonruter added this to Definition in v1.0 Jul 11, 2018

@postphotos

This comment has been minimized.

Copy link
Contributor

commented Jul 19, 2018

Hi @amedina, Thanks for this! I've wrote a quick user story for this ticket. We should be able to add this to our v1.0 stable release.

@postphotos postphotos moved this from Definition to To do in v1.0 Jul 19, 2018

@hellofromtonya hellofromtonya self-assigned this Jul 23, 2018

@hellofromtonya hellofromtonya moved this from To do to In progress in v1.0 Jul 29, 2018

@hellofromtonya hellofromtonya moved this from In progress to Ready for review in v1.0 Jul 30, 2018

@hellofromtonya hellofromtonya moved this from Ready for review to Ready for QA in v1.0 Jul 30, 2018

@hellofromtonya hellofromtonya moved this from Ready for QA to Ready for Merging in v1.0 Jul 30, 2018

@hellofromtonya hellofromtonya moved this from Ready for Merging to Ready for QA in v1.0 Jul 30, 2018

@kienstra

This comment has been minimized.

Copy link
Collaborator

commented Aug 3, 2018

Confirmed, Moving To "Ready For Merging"

This look good. "Native" now appears at the top of the "Template Mode" field.

I'm moving this to "Ready For Merging."

amp-settings

@kienstra kienstra moved this from Ready for QA to Ready for Merging in v1.0 Aug 3, 2018

@kienstra kienstra moved this from Ready for Merging to In Production in v1.0 Dec 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.