Making WordPress.org

Opened 2 years ago

Closed 21 months ago

#5977 closed defect (bug) (fixed)

Move the position of terms description at the top of the commercial themes page

Reported by: bijayyadav's profile bijayyadav Owned by: dufresnesteven's profile dufresnesteven
Milestone: Priority: normal
Component: Theme Directory Keywords: has-patch needs-design-feedback needs-copy-review
Cc:

Description

While I was checking the terms of the commercial themes page, I was not able to figure it out at first. I had to scroll the whole page to find it.

I think, it's better to move the description to the top of the page and it will be easy to access.

See the screenshot how it looks like after moving it to the top section.

Attachments (4)

Screenshot_7.jpg (199.6 KB) - added by bijayyadav 2 years ago.
commercial-themes-page-description-position-changed.patch (3.2 KB) - added by bijayyadav 2 years ago.
5977.diff (1.3 KB) - added by dufresnesteven 2 years ago.
Screen Shot 2022-03-25 at 11.39.24 AM.png (1.1 MB) - added by dufresnesteven 2 years ago.

Download all attachments as: .zip

Change History (19)

#1 @kafleg
2 years ago

Hello @bijayyadav

Thank you for creating this ticket and a warm welcome to meta.
I was feeling something similar that some theme authors didn't follow these terms while submitting the request to add their site.

Instead of adding terms at the top, we can add a link at the top section scroll down to the detailed terms section.

Thank you

This ticket was mentioned in PR #45 on WordPress/wordpress.org by bijayyadav.


2 years ago
#2

While I was checking the terms of the commercial themes page, I was not able to figure it out at first. I had to scroll the whole page to find it.

I think, it's better to move the description to the top of the page and it will be easy to access.

https://meta.trac.wordpress.org/ticket/5977

#3 @dufresnesteven
2 years ago

Hi @bijayyadav,

I'm assuming that it's below the content because the main use case is that users are looking for commercial themes and not looking to add theirs to the directory. Assuming that was the original intention, would it make sense to have a short callout and link to the section below the content instead of all the details?

Maybe something like:

"Looking to add your theme to this list? Read more about the requirements."

#4 @bijayyadav
2 years ago

Hi @dufresnesteven

Makes sense. Something that links to the requirement list from the top would be better so that users can navigate to the list easily.

@dufresnesteven
2 years ago

#5 @dufresnesteven
2 years ago

  • Keywords needs-design-feedback needs-copy-review added

#6 @kafleg
2 years ago

@dufresnesteven That is a perfect way to do that. It is my recommendation as well.

This ticket was mentioned in Slack in #meta by dufresnesteven. View the logs.


2 years ago

#8 @costdev
2 years ago

I think this would be clearer:

Want to see your theme in this list? View the requirements.

  • "Looking to" is not commonly used in all English speaking areas.
  • "Read more" is non-descriptive text.

#9 @dd32
2 years ago

It's worth noting, that this is not a list of themes, this is a list of theme companies.

Perhaps instead:

Want to see your company on this list? View the requirements.

This existing sentence is.. questionable though:

While our directory is full of fantastic themes, sometimes people want to use something that they know has support behind it, and don’t mind paying for that.

Are you trying to tell me that free themes don't have support? or that I'll definitely get support from these companies?

That might explain why some of the Theme support tickets have been all "I bought it, but they still refuse to talk to me"

#10 @dufresnesteven
2 years ago

  • Owner set to dufresnesteven
  • Resolution set to fixed
  • Status changed from new to closed

In 11718:

Theme: Link to commercial theme requirements at top of page.

Props bijayyadav, kafleg, costdev, dd32
Fixes #5977

#11 @dd32
2 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

@dufresnesteven In [11718]:
Either it should be a single string:
printf( __( 'Want to see your company on this list? <a href="%s">View the requirements</a>' ....
or, the second string shouldn't contain the HTML:
echo '<a href="#themeRequirements">' . __( 'View the requirements' ....

Personally I'd go for the first option.

dd32 commented on PR #45:


2 years ago
#12

Alternative direction taken in https://meta.trac.wordpress.org/ticket/5977

#13 @dufresnesteven
2 years ago

In 11719:

Theme: Update use of printf in commercial themes.

See #5977

#14 @kafleg
2 years ago

Thank you all :)

#15 @dufresnesteven
21 months ago

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.