WordPress.org

Make WordPress Themes

Opened 2 years ago

Closed 2 years ago

#26236 closed theme (not-approved)

THEME: Blackboard Lite – 1.0.3

Reported by: academiathemes Owned by: TimothyBlynJacobs
Priority: new theme Keywords: theme-blackboard-lite
Cc: academiathemes@…

Description

Blackboard Lite - 1.0

Blackboard Lite is a simplified and free version of the Blackboard WordPress Theme developed for schools, universities, kindergartens, organizations, companies, etc.

Theme URL - http://www.academiathemes.com/themes/blackboard/
Author URL - http://www.academiathemes.com/

SVN - https://themes.svn.wordpress.org/blackboard-lite/1.0
ZIP - https://wordpress.org/themes/download/blackboard-lite.1.0.zip?nostats=1

History:

Ticket Summary Status Resolution Owner
#26236 THEME: Blackboard Lite – 1.0.3 closed not-approved TimothyBlynJacobs

(this ticket)


https://themes.svn.wordpress.org/blackboard-lite/1.0/screenshot.png

Change History (9)

#1 @themetracbot
2 years ago

  • Summary changed from THEME: Blackboard Lite – 1.0 to THEME: Blackboard Lite – 1.0.2

Blackboard Lite - 1.0.2

Blackboard Lite is a simplified and free version of the Blackboard WordPress Theme developed for schools, universities, kindergartens, organizations, companies, etc.

Theme URL - http://www.academiathemes.com/themes/blackboard/
Author URL - http://www.academiathemes.com/

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

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=blackboard-lite/1.0&new_path=blackboard-lite/1.0.2

History:

Ticket Summary Status Resolution Owner
#26236 THEME: Blackboard Lite – 1.0.3 closed not-approved TimothyBlynJacobs

(this ticket)


https://themes.svn.wordpress.org/blackboard-lite/1.0.2/screenshot.png

#2 @themetracbot
2 years ago

  • Summary changed from THEME: Blackboard Lite – 1.0.2 to THEME: Blackboard Lite – 1.0.3

Blackboard Lite - 1.0.3

Blackboard Lite is a simplified and free version of the Blackboard WordPress Theme developed for schools, universities, kindergartens, organizations, companies, etc.

Theme URL - http://www.academiathemes.com/themes/blackboard/
Author URL - http://www.academiathemes.com/

SVN - https://themes.svn.wordpress.org/blackboard-lite/1.0.3
ZIP - https://wordpress.org/themes/download/blackboard-lite.1.0.3.zip?nostats=1

Diff with previous version: https://themes.trac.wordpress.org/changeset?old_path=blackboard-lite/1.0.2&new_path=blackboard-lite/1.0.3

History:

Ticket Summary Status Resolution Owner
#26236 THEME: Blackboard Lite – 1.0.3 closed not-approved TimothyBlynJacobs

(this ticket)


https://themes.svn.wordpress.org/blackboard-lite/1.0.3/screenshot.png

#3 follow-up: @themetracbot
2 years ago

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

This ticket was mentioned in Slack in #themereview by timothybjacobs. View the logs.


2 years ago

#5 in reply to: ↑ 3 @academiathemes
2 years ago

Replying to themetracbot:
Hello Timothy,
Thank you for taking on the review for our theme.
I'll be glad to work with you quickly on fixing any issues that you might find with it.
Dumitru

#6 @TimothyBlynJacobs
2 years ago

  • Resolution set to not-approved
  • Status changed from reviewing to closed

Hi academiathemes,

My name is Timothy. I'm going to be reviewing your theme.

At this time your theme isn't ready for inclusion in the WordPress themes directory. I've outlined a number of things that you must fix, and recommendations that you should fix. If something isn't clear let me know. This might not be all of the issues, but they are the ones that stood out to me upon first look.

Required

  • It seems like the header is not being loaded on the posts page. Can you confirm?
  • .screen-reader-text css class is needed in your theme css. See: the Codex for an example implementation.
  • Theme needs to obey the front page option. If the homepage slider is turned on, the front page will display the slider but with the list of blog posts underneath it. It should display the slider with the front page beneath it.
  • When saving the use in slider option you must first check if the value isset. Also esc_attr is not valid sanitization here. It is for escaping output. Since this is just a checkbox you should call update_post_meta with a value of true if academia_post_featured is set, and if not, call delete_post_meta for that same option key.
  • Don’t call wp_reset_query in footer.php
  • When registering sidebars and menus, you must translate the sidebar and menu names

Recommended

  • Add a recommended size for the logos or constrain their width to fit
  • An extra wide image breaks the content area.
  • You should add a read more link to posts that have more tags.
  • Featured images aren’t displayed on single post pages
  • When the static front page is enabled, menus in the menu widget are not nested underneath each other.
  • Set a resonable upper bound on the total number of slideshow images displayed, and prevent negative numbers. -1 would display all items.
  • Setting up the slider is a bit confusing. I’d recommend making the slider possible to use without using the home page template.
  • In header.php when calling wp_nav_menu you provided menu_class and menu_id keys twice.

Notes

  • Consider reducing the delay on hiding the menu items
  • Consider adding styling for inputs with a type of button

After all of the required issues are fixed, you can upload a new version of your theme and I will take another look at your theme.

#7 @jcastaneda
2 years ago

  • Resolution not-approved deleted
  • Status changed from closed to reopened

#8 @jcastaneda
2 years ago

  • Status changed from reopened to reviewing

#9 @karmatosed
2 years ago

  • Resolution set to not-approved
  • Status changed from reviewing to closed

I am closing this ticket as you've not responded to the review. In future please respond, even to say you need more time within 7 days.

Note: See TracTickets for help on using tickets.