CommentFileSizeAuthor
#8 i2614800-8.patch1.32 KBiamDamon
#3 i2614800-3.patch413 bytesiamDamon
Screen Shot 2015-11-13 at 13.43.04.png50.04 KBiamDamon

Comments

iamDamon created an issue. See original summary.

iamDamon’s picture

iamDamon commented
Assigned: Unassigned » iamDamon
Issue summary: View changes
iamDamon’s picture

iamDamon commented
Assigned: iamDamon » Unassigned
StatusFileSize
new i2614800-3.patch413 bytes

Fix for datetime

attiks’s picture

attiks commented
Status: Active » Needs review
Issue tags: -datetime +Needs subsystem maintainer review

Looks good to me, not sure if the css should be split in 2

darketaine’s picture

darketaine commented
Status: Needs review » Needs work

In terms of consistency I think that what is really missing here is the form-item wrapper and the problem is not CSS. All other fields don't have margin in their external wrappers.

darketaine’s picture

darketaine commented
Component: CSS » Classy theme

It is more of a Classy TWIG issue (datetime-wrapper.html.twig)

iamDamon’s picture

iamDamon commented
Assigned: Unassigned » iamDamon
iamDamon’s picture

iamDamon commented
Assigned: iamDamon » Unassigned
Status: Needs work » Needs review
StatusFileSize
new i2614800-8.patch1.32 KB

Added form-item in TWIG, should be fixed.

Status: Needs review » Needs work

The last submitted patch, 8: i2614800-8.patch, failed testing.

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

emma.maria’s picture

emma.maria
she/her
commented
Status: Needs work » Closed (duplicate)

This issue is a duplicate of ... #2643224: Form fields with the .container-inline class do not have spacing around.

Which in turn is a bug issue that would be fixed in this task issue... #2643224: Form fields with the .container-inline class do not have spacing around

As more progress has been made in the first issue, I'm marking this one as Closed (duplicate).