Making WordPress.org

Opened 5 years ago

Closed 4 years ago

Last modified 4 years ago

#4383 closed defect (bug) (fixed)

Plugin 'Reviews' tab should provide more information to logged-out users

Reported by: dannycooper's profile DannyCooper Owned by: dannycooper's profile DannyCooper
Milestone: Priority: high
Component: Plugin Directory Keywords: needs-patch
Cc:

Description

To a logged out user, it doesn't appear there is any way to review a plugin:

https://i.imgur.com/IIRQ2Gm.png

The Support tab is a bit clearer:

https://i.imgur.com/V7xB0II.png

However, if a plugin already has a lot of support threads, then that tab is unclear too as the message gets pushed down to the bottom. Example: https://wordpress.org/support/plugin/olympus-google-fonts/

It would be useful if these messages showed at the top for logged-out users.

Attachments (3)

Screen Shot 2019-12-05 at 8.55.30 AM.png (105.0 KB) - added by dufresnesteven 5 years ago.
Plugin has no review: User not logged In.
Screen Shot 2019-12-04 at 4.16.02 PM.png (108.1 KB) - added by dufresnesteven 5 years ago.
Plugin has no reviews: User logged In.
4383.diff (1.9 KB) - added by dufresnesteven 5 years ago.

Download all attachments as: .zip

Change History (13)

#1 @pratikgandhi
5 years ago

  • Keywords dev-feedback close added
  • Resolution set to worksforme
  • Status changed from new to closed

Hello @DannyCooper

Thank you for the ticket.

I just checked the plugin's review tab and I think it's now updated. I can see the reviews without login and with login in this tab easily. Hope it is visible to you as well.

Thank you

#2 @DannyCooper
5 years ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

Hi @pratikgandhi,

This isnt fixed, please check a plugin that has no reviews when you are not logged in:

https://wordpress.org/plugins/design-experiments/#reviews

The tab is completely blank.

#3 @pratikgandhi
5 years ago

  • Keywords needs-patch added; dev-feedback close removed
  • Priority changed from normal to high

Hi @DannyCooper

yes, There should be some message like 'This plugin has not been rated yet, Add my review'.

Thanks

#4 @DannyCooper
5 years ago

I'm not familiar enough with the .org codebase to submit a patch, but I think something as simple as this would work:

https://i.imgur.com/rWlcnAR.png

@dufresnesteven
5 years ago

Plugin has no review: User not logged In.

@dufresnesteven
5 years ago

Plugin has no reviews: User logged In.

@dufresnesteven
5 years ago

#5 @dufresnesteven
5 years ago

  • Owner set to dufresnesteven
  • Status changed from reopened to assigned

Patch attachment:4383.diff.


I've submitted a patch to add a message on the left side of the reviews tab for the following scenarios:


The Plugin has no reviews and the user is logged out.

Plugin has no review: User not logged In.



The Plugin has no reviews and the user is logged in.

Plugin has no reviews: User logged In.



Notes:

  • I made use of our newer "notice" styles for the messaging

Deployment Instructions:

  • Need to recompile wporg-plugins css.
Last edited 5 years ago by dufresnesteven (previous) (diff)

#6 @dufresnesteven
5 years ago

  • Owner changed from dufresnesteven to DannyCooper

Did I interpret your expected change properly?

Last edited 5 years ago by dufresnesteven (previous) (diff)

#7 @DannyCooper
5 years ago

Yes, it looks like you did @dufresnesteven :-)

#8 @dd32
4 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 9652:

Plugin Directory: Add a login link to add reviews, and a notice when there's no reviews for the plugin.

This differs from the ticket patches slightly, in that it adds the login link to the sidebar rather than the notice, for consistency when a plugin has existing reviews.

Props dufresnesteven.
Fixes #4383.

#9 @pedromendonca
4 years ago

Hi @dd32,

The Log in to WordPress.org string should have capital P.
The trac is autocorrecting the capital P, so I can't report it correcly

Please see ticket https://meta.trac.wordpress.org/ticket/5124

#10 @Otto42
4 years ago

In 9654:

Plugin Directory: Capital P correction for [9652]. See #4383.

Note: See TracTickets for help on using tickets.