[drewbul]

Hello

First of all I’d like to thank you for the great job. I like your Verbosa theme very much. I will definitely donate or buy support, after the website is ready. Here’s some feedback, which, I hope, can make the theme even better.

I have performed HTML/CSS validation of my website running on WordPress 4.7 and Verbosa theme 0.9.7, and here are the errors that were found:

/wp-content/themes/verbosa/resources/fonts/fontfaces.css
line 27:
wdith: 1em;
-> -> ->
width: 1em;

line 212:
.socials a[href*="500px.com"]:before, .socials a.500px:before { content: '\e80e'; }
-> -> ->
/*.socials a[href*="500px.com"]:before, .socials a.500px:before { content: '\e80e'; }*/

line 221:
.socials a[href*="99designs.com"]:before, .socials a.99designs:before { content: '\e817'; }
-> -> ->
/*.socials a[href*="99designs.com"]:before, .socials a.99designs:before { content: '\e817'; }*/

In CSS2, property value that begins with a digit — the first character should be escaped, but I just commented out those lines, since I don’t use those social networks, and probably never will.

line 229:
.socials a[href*="creativecommons.org"]:before, .socials a.cc:beforem, .socials a.creativecommons { content: '\e820'; }
-> -> ->
.socials a[href*="creativecommons.org"]:before, .socials a.cc:before, .socials a.creativecommons { content: '\e820'; }

/wp-content/themes/verbosa/style.css

line 110:
text-rendering: optimizeLegibility;
-> -> ->
/* text-rendering: optimizeLegibility; */

Looks OK, but somehow the validator didn’t like it, and for me it was not important.

/wp-content/themes/verbosa/includes/core.php

line 53:
$wp_logo = str_replace( 'class="custom-logo-link"', 'id="logo" class="custom-logo-link" alt="'.esc_attr( get_bloginfo( 'name', 'display' ) ).'" title="'.esc_attr( get_bloginfo( 'name', 'display' ) ).'"', get_custom_logo() );
-> -> ->
$wp_logo = str_replace( 'class="custom-logo-link"', 'id="logo" class="custom-logo-link" title="'.esc_attr( get_bloginfo( 'name', 'display' ) ).'"', get_custom_logo() );

According to the validator and various articles, “alt” attribute cannot be applied to “a” tag.

==============================

And here’s not an error exactly, but some considerable improvement in my case. I had these problem when scrolling down a page with an expanded big submenu (17 items) and a small article:

– sometimes the scrolling down never stopped — the page automatically jumped to the top and the submenu unfolded again;

– sometimes it was not possible to scroll down at all, so the bottom part of the big submenu was not visible.

I solved it with custom CSS:

#sidebar {
 max-height: 100% !important;
 overflow-y: auto !important;
}

Website: new.ate.org.ua

• This topic was modified 8 years ago by drewbul.
• This topic was modified 8 years ago by drewbul.

[Author]

Posts