Skip to content

Debugger: add link to the old modules list.#6770

Merged
eliorivero merged 1 commit intomasterfrom
add/module-list-debug
Mar 27, 2017
Merged

Debugger: add link to the old modules list.#6770
eliorivero merged 1 commit intomasterfrom
add/module-list-debug

Conversation

@jeherve
Copy link
Copy Markdown
Member

@jeherve jeherve commented Mar 27, 2017

  • Include that link only if user has access to the list.
  • Create new section, separated by an horizontal line and including a heading, so the link is easier to locate on the page.

Before:

screen shot 2017-03-27 at 20 14 48

After:

screen shot 2017-03-27 at 20 14 15

If you have any feedback on wording or anything, please let me know! I think the wording could be improved, but I'm not really sure how to phrase this.

- Include that link only if user has access to the list.
- Create new section, separated by an horizontal line and including a heading, so the link is easier to locate on the page.
@jeherve jeherve added Admin Page React-powered dashboard under the Jetpack menu [Status] Needs Review This PR is ready for review. Enhancement Changes to an existing feature — removing, adding, or changing parts of it labels Mar 27, 2017
@jeherve jeherve added this to the Settings UI milestone Mar 27, 2017
@jeherve jeherve self-assigned this Mar 27, 2017
Copy link
Copy Markdown
Contributor

@dereksmart dereksmart left a comment

Choose a reason for hiding this comment

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

LGTM

@dereksmart dereksmart added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Mar 27, 2017
Copy link
Copy Markdown
Contributor

@zinigor zinigor left a comment

Choose a reason for hiding this comment

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

LGTM!

@eliorivero eliorivero merged commit 34cf2c4 into master Mar 27, 2017
@eliorivero eliorivero deleted the add/module-list-debug branch March 27, 2017 20:59
@eliorivero eliorivero removed the [Status] Ready to Merge Go ahead, you can push that green button! label Mar 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Admin Page React-powered dashboard under the Jetpack menu Enhancement Changes to an existing feature — removing, adding, or changing parts of it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants