WordPress.org

Making WordPress.org

Opened 8 months ago

Last modified 6 weeks ago

#2854 new defect

On the Advanced View of a plugin the tabs list is missing

Reported by: ramiy Owned by:
Milestone: Plugin Directory v3 - Future Priority: normal
Component: Plugin Directory Keywords: good-first-bug has-patch
Cc:

Description

The tabs list navigation (Details, Reviews, Installation, Support, Development) is missing when the Advanced View is displayed.

Attachments (4)

AkismetDetails.png (274.6 KB) - added by ramiy 8 months ago.
AkismetAdvancedView.png (226.7 KB) - added by ramiy 8 months ago.
2854.diff (2.5 KB) - added by audrasjb 3 months ago.
Add plugin main tabs in advanced view page
2854.2.diff (2.6 KB) - added by audrasjb 3 months ago.
Add missing echo instruction to the permalink :P

Download all attachments as: .zip

Change History (8)

@ramiy
8 months ago

#1 @tellyworth
3 months ago

  • Keywords needs-patch good-first-bug added

@audrasjb
3 months ago

Add plugin main tabs in advanced view page

#2 @audrasjb
3 months ago

  • Keywords has-patch added; needs-patch removed

Hi,
Here is a patch to add plugin main tabs in advanced view.
I added plugin permalink to the existing anchors for advanced view only.

Cheers,
Jb

@audrasjb
3 months ago

Add missing echo instruction to the permalink :P

#3 @joostdevalk
6 weeks ago

Patch looks good to me, @SergeyBiryukov or @otto42 can you have a look and commit? :)

#4 @Otto42
6 weeks ago

This doesn't seem like the right way to me. I'd rather get rid of the Advanced view and move its contents somewhere else.

Also, tabs are intentionally missing from the Advanced view. The Advanced view was only created last minute because of internal changes. We need to come up with a bigger and better plan than simply monkey patching this one.

So for now, I say wontfix until we have a better idea.

Last edited 6 weeks ago by Otto42 (previous) (diff)
Note: See TracTickets for help on using tickets.