Dartik is a dark version of Bartik - Drupal 7 core's default theme.
It is basically a copy of all color styles from Bartik leaving all other styles untouched, to be inherited from the original theme.

  • Fully compatible with color module.
  • Contains all RTL overrides and images needed for right-to-left sites.
  • Tested as admin theme too.

The idea for a dark scheme for Bartik came up while we were thinking of improvements to the color schemes available at that time in #762468: Fix up the color schemes for Bartik but as Jen (Bartik's creator and mainrainer) mention here and here - "Bartik won't work light text on dark background. Ever. Unless someone makes a contrib theme like that.... :P" - so here it is.

Attached is a screen shot.

CommentFileSizeAuthor
#6 dartik.patch3.39 KBtsi
dartik.png42.99 KBtsi

Comments

tsi’s picture

tsi commented

And here's a link to the sandbox project - http://drupal.org/sandbox/tsi/1114098

tsi’s picture

tsi commented

Hi guys,
any news on that ?

sreynen’s picture

sreynen commented
Status: Active » Needs review

Changing the status to "needs review" is an important part of the instructions, because reviewers tend to look only at issues in that status.

tsi’s picture

tsi commented

my bad, thanks.

dcmouyard’s picture

dcmouyard commented
Status: Needs review » Needs work

This theme follows the coding standards pretty well, but I do have two minor nitpicks:

  • CSS properties should be listed in alphabetical order. (CSS coding standards)
  • Use shorthand for hex colors whenever possible. #222 instead of #222222

The major issue, however, is that this theme is not accessible due to color contrast. I created an issue at #1140046: All text output should should meet minimum contrast guidelines

Use the Style Guide module to preview the most common Drupal output in this theme. There are several occurances of light text on a white background and black text on a dark background.

tsi’s picture

tsi commented
Status: Needs work » Needs review
StatusFileSize
new dartik.patch3.39 KB

Thank's for the review,

I've answered about the accessibility issue on #1140046: All text output should should meet minimum contrast guidelines.

Regarding CSS properties order and shorthands, I've modified my style sheets, patch with all changes attached.

Commit - http://drupalcode.org/sandbox/tsi/1114098.git/commit/2782112

sreynen’s picture

sreynen commented
Status: Needs review » Reviewed & tested by the community

I'm marking this as reviewed. I haven't found any new issues and the only issues still open has been partially resolved while tsi awaits more details. It seems like something that may take a while to work out, and not something that needs to keep this from becoming a full project.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Reviewed & tested by the community » Fixed

Thanks for your contribution, tsi! Welcome to the community of contributors on drupal.org.

I've granted you the git vetted user role which will let you promote this to a full project and also create new projects as either sandbox or "full" projects depending on which you feel is best.

Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

tsi’s picture

tsi commented

Thanks

sarah_p’s picture

sarah_p commented
Component: new project application » theme

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.