WordPress.org

Make WordPress Themes

Opened 6 months ago

Closed 6 months ago

#14770 closed theme (live)

THEME: Narcissus - 1.0.1

Reported by: DaJoker29 Owned by: rohitink
Priority: new theme Keywords: theme-narcissus
Cc: dj729@…, rohitink

Description

Narcissus - 0.3.1

A simple wordpress theme based on modularity and simplicity

Theme URL - https://github.com/DaJoker29/narcissus
Author URL - http://zerodaedalus.com

SVN - http://themes.svn.wordpress.org/narcissus/0.3.1
ZIP - http://wordpress.org/themes/download/narcissus.0.3.1.zip?nostats=1

History:

Ticket Summary Status Resolution Owner
#14770 THEME: Narcissus - 1.0.1 closed live rohitink

(this ticket)


https://themes.svn.wordpress.org/narcissus/0.3.1/screenshot.png

Change History (12)

comment:1 tskk6 months ago

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

comment:2 tskk6 months ago

  • Cc emiluzelac rohitink added

Emil, this one too.

comment:3 emiluzelac6 months ago

  • Owner changed from tskk to rohitink

comment:4 tskk6 months ago

Thanks Emil.

comment:5 follow-up: rohitink6 months ago

  1. Post title & content containing very long words overflow their container.
  1. Recommendation: Sub Menu items of a custom menu in sidebar are hitting the eye too much. Not so easy to read. I recommend you to change the color a little bit, or add a slight text-shadow to it. Other sidebar links are also not readable, so a dark textshadow can really help, just like you have added to post titles.
  1. When wordpress loads up the default menu in the Navbar, the dropdown is unreadable as it directly overlaps.
  1. Remove the readme.md file.
  1. Add license information about the resources like frameworks,images,fonts,js,etc used in the theme.
  1. Don't include the style.css file from the header.php file. It is already being enqueued in your functions.php. Secondly, wp_head(); should be called just before </head>

Another recommendation: Create and use a nicer screenshot, if you want others to be attracted towards your theme.

Please fix these issues and re-upload the theme.

comment:6 in reply to: ↑ 5 ; follow-up: DaJoker296 months ago

Alright, cool. I'll get to work on these.

I'm guessing I'm the one you mentioned on the mailing list about readability issues. I'll admit I'm pretty new and not very good at this; more of a programmer than I am a designer. I could definitely use any feedback/criticism/advice you have to offer so that I can improve.

comment:7 themetracbot6 months ago

  • Summary changed from THEME: Narcissus - 0.3.1 to THEME: Narcissus - 1.0.1

comment:8 in reply to: ↑ 6 rohitink6 months ago

Replying to DaJoker29:

I'm guessing I'm the one you mentioned on the mailing list about readability issues. I'll admit I'm pretty new and not very good at this; more of a programmer than I am a designer. I could definitely use any feedback/criticism/advice you have to offer so that I can improve.

Yes, please don't mind that. I am sure you will definitely enhance your design skills in the future. You could browse CSS Galleries on the internet to understand what are some good web designs. Just take inspiration, do not copy.

I am reviewing your new version right now.

comment:9 rohitink6 months ago

I think you didn't understand the license part. You need to create a readme.txt of credits.txt file and add license information about all the fonts/javascripts/php files you have used. Please look into the readme.txt file in any one of my themes, and see how license information is added.

Re-upload the theme with all the license information properly added. Ask if you need help.

comment:11 rohitink6 months ago

  • Cc emiluzelac removed
  • Status changed from reviewing to approved

All Issues with this theme have been fixed. Theme passed all Unit tests. Code has been checked. There are no visual issues. Approving the theme now.

comment:12 chipbennett6 months ago

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