Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
Seven theme
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
28 Jun 2013 at 14:27 UTC
Updated:
29 Jul 2014 at 22:35 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
emma.mariaI have tidied up the text by aligning everything with the first header. RTL styles have also been added. However a class needs to be applied to this page to override the default installer page styles.
Comment #2
mordonez commentedI think it's only necessary align the "ul" and decreasing the site-name space.
Comment #3
ckrinaLast patch works fine.
Comment #4
ckrinaSorry, reviewed on RTL and still some changes needed.
Attaching screenshots.
Comment #5
aspilicious commented0 or 0em sounds the same. Or isn't it? I don't think we need the rtl version than as it is the same. If not explain me what the difference is between 0 en 0em.
Comment #6
ckrinaComment #7
rteijeiro commented[dir=rtl]must be[dir="rtl"]according to #2015789: Remove language_css_alter() (RTL stylesheets) in favor of HTML 'dir' attributeI'm not sure about using
#content. We should use less specific selectors and classes when possible.Comment #8
emma.mariaComment #9
emma.mariaI have changed all of the [dir=rtl] to [dir="rtl"]. I have removed the #content from the ul selector as requested and also the body.install-page as it is not needed here.
Comment #10
rteijeiro commentedIt seems good.
Comment #11
lewisnyman+1 RTBC
Comment #12
dries commentedCommitted to 8.x. Thanks!