Dartik

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.

Comment	File	Size	Author
#6	dartik.patch	3.39 KB	tsi
	dartik.png	42.99 KB	tsi

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

tsi CreditAttribution: tsi commented 2 April 2011 at 15:04

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

Comment #2

tsi CreditAttribution: tsi commented 14 April 2011 at 07:09

Hi guys,
any news on that ?

Comment #3

sreynen CreditAttribution: sreynen commented 22 April 2011 at 04:55

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.

Comment #4

tsi CreditAttribution: tsi commented 26 April 2011 at 19:28

my bad, thanks.

Comment #5

dcmouyard CreditAttribution: dcmouyard commented 27 April 2011 at 15:13

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.

Comment #6

tsi CreditAttribution: tsi commented 30 April 2011 at 09:20

Status:

Needs work

» Needs review

File	Size
dartik.patch	3.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

Comment #7

sreynen CreditAttribution: sreynen commented 30 April 2011 at 17:26

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.

Comment #8

greggles

he/him

English

Denver, Colorado, USA

CreditAttribution: greggles commented 3 May 2011 at 17:48

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.