WordPress.org

Make WordPress Themes

Opened 3 months ago

Closed 5 weeks ago

#44606 closed theme (live)

THEME: Veroxa – 1.0.2

Reported by: mightywp Owned by: meissudeep
Priority: new theme Keywords: theme-veroxa
Cc: support@…

Description

Veroxa - 1.0.0

Veroxa is a fast WordPress Magazine suitable for blogs, news, publishing and magazine sites.

Theme URL - https://mightywp.com/themes/veroxa/
Author URL - https://mightywp.com/

Trac Browser - https://themes.trac.wordpress.org/browser/veroxa/1.0.0

SVN - https://themes.svn.wordpress.org/veroxa/1.0.0
ZIP - https://wordpress.org/themes/download/veroxa.1.0.0.zip?nostats=1

History:

Ticket Summary Status Resolution Owner
#44606 THEME: Veroxa – 1.0.2 closed live meissudeep

(this ticket)

#46478 THEME: Veroxa – 1.0.3 closed live themetracbot


https://themes.svn.wordpress.org/veroxa/1.0.0/screenshot.png
Theme Check Results:

  • RECOMMENDED: No reference to add_theme_support( "custom-header", $args ) was found in the theme. It is recommended that the theme implement this functionality if using an image for the header.
  • RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.

Change History (10)

#1 @joyously
8 weeks ago

Here is some user feedback to consider before your review.

  • The menu needs to have a fallback for when no menu is selected. The default value of fallback_cb parameter of wp_nav_menu() works really well, and it's core functionality.
  • Custom Menu widgets show the links all at the same level. (menu structure is lost)
  • A post with no title has no link to the single post page, from an archive.
  • Tags that have been visited are blue on blue (invisible).
  • The Calendar widget has the headings centered and the numbers left, so it does not line up well.
  • The page links have a label (Pages), but no numbers. And they are set as screen reader text, so will they be visible once output?
  • <h6> is quite small. How is that a heading?
  • <q> tag is unstyled. (default removed)
  • Floats are not being cleared properly after the content area on Posts.
  • You might want to hide the menu, comment form, and widget areas for the print styles.
  • The Search widget input has the text as white on white while I'm typing.
  • Search results should not show the author and date of static Pages.
  • The post author is not linked to the author archive.
  • Theme should support the gallery default of 3 columns.
  • Customizer, Footer Copyright - The "text on the right" option is showing the HTML in the preview.

#2 @meissudeep
7 weeks ago

  • Owner set to meissudeep
  • Status changed from new to reviewing

Hello @mightywp

I am taking your theme, will be back soon with my review.

#3 @meissudeep
7 weeks ago

Hello @mightywp

Fix the following issue and upload your theme, then your theme will be ready to set live.

ISSUES

  • Untranslated string is found check header.php L-84. check other files as well.
  • Didn't see any languages folder check functions.php L-26.
  • Use proper way of integrating google font, you can check default themes for more informations.

Please respond within 7 days. If there is no response for 7 days, ticket will be closed as not-approved.

#4 @themetracbot
7 weeks ago

  • Summary changed from THEME: Veroxa – 1.0.0 to THEME: Veroxa – 1.0.1

Veroxa - 1.0.1

Veroxa is a fast WordPress Magazine suitable for blogs, news, publishing and magazine sites.

Theme URL - https://mightywp.com/themes/veroxa/
Author URL - https://mightywp.com/

Trac Browser - https://themes.trac.wordpress.org/browser/veroxa/1.0.1

SVN - https://themes.svn.wordpress.org/veroxa/1.0.1
ZIP - https://wordpress.org/themes/download/veroxa.1.0.1.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=veroxa/1.0.0&new_path=veroxa/1.0.1

History:

Ticket Summary Status Resolution Owner
#44606 THEME: Veroxa – 1.0.2 closed live meissudeep

(this ticket)

#46478 THEME: Veroxa – 1.0.3 closed live themetracbot


https://themes.svn.wordpress.org/veroxa/1.0.1/screenshot.png
Theme Check Results:

  • RECOMMENDED: No reference to add_theme_support( "custom-header", $args ) was found in the theme. It is recommended that the theme implement this functionality if using an image for the header.
  • RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.

#5 @mightywp
7 weeks ago

Hi @meissudeep,

I think I fixed all the errors that you mentioned.

Thanks!

#6 @meissudeep
6 weeks ago

Oho I miss the notification. Will check and let you know soon :)

#7 @meissudeep
6 weeks ago

Hello @mightywp

Everything looks cool just remove the empty folder languages and commented code's related to google font form function.php. check other files as well.

Early escaping is done here at archive.php L-26 & L-36. Recommend you to do escaping while you output's the content.

Other things looks fine.

#8 @mightywp
6 weeks ago

Hi @meissudeep,

Removed language and comment folder.

Also removed description code because I've decided to go another path later on adding an option to the customizer.

#9 @themetracbot
6 weeks ago

  • Summary changed from THEME: Veroxa – 1.0.1 to THEME: Veroxa – 1.0.2

Veroxa - 1.0.2

Veroxa is a fast WordPress Magazine suitable for blogs, news, publishing and magazine sites.

Theme URL - https://mightywp.com/themes/veroxa/
Author URL - https://mightywp.com/

Trac Browser - https://themes.trac.wordpress.org/browser/veroxa/1.0.2

SVN - https://themes.svn.wordpress.org/veroxa/1.0.2
ZIP - https://wordpress.org/themes/download/veroxa.1.0.2.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=veroxa/1.0.1&new_path=veroxa/1.0.2

History:

Ticket Summary Status Resolution Owner
#44606 THEME: Veroxa – 1.0.2 closed live meissudeep

(this ticket)

#46478 THEME: Veroxa – 1.0.3 closed live themetracbot


https://themes.svn.wordpress.org/veroxa/1.0.2/screenshot.png
Theme Check Results:

  • RECOMMENDED: No reference to add_theme_support( "custom-header", $args ) was found in the theme. It is recommended that the theme implement this functionality if using an image for the header.
  • RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.

#10 @meissudeep
5 weeks ago

  • Resolution set to live
  • Status changed from reviewing to closed

Hello @mightywp

Everything looks fine, setting this theme to live. All the best for your theme.

congratulations

Note: See TracTickets for help on using tickets.