WordPress.org

Making WordPress.org

Opened 12 months ago

Closed 2 days ago

Last modified 41 hours ago

#4383 closed defect (fixed)

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

Reported by: DannyCooper Owned by: 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 4 months 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 4 months ago.
Plugin has no reviews: User logged In.
4383.diff (1.9 KB) - added by dufresnesteven 4 months ago.

Download all attachments as: .zip

Change History (13)

#1 @pratikgandhi
9 months 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
9 months 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
9 months 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
9 months 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
4 months ago

Plugin has no review: User not logged In.

@dufresnesteven
4 months ago

Plugin has no reviews: User logged In.

@dufresnesteven
4 months ago

#5 @dufresnesteven
4 months 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 4 months ago by dufresnesteven (previous) (diff)

#6 @dufresnesteven
4 months ago

  • Owner changed from dufresnesteven to DannyCooper

Did I interpret your expected change properly?

Last edited 4 months ago by dufresnesteven (previous) (diff)

#7 @DannyCooper
4 months ago

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

#8 @dd32
2 days 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
42 hours 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
41 hours ago

In 9654:

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

Note: See TracTickets for help on using tickets.