Vertical alignment of text in drag and drop cells isn't consistent. I propose that we add a top and bottom padding of 10px.

An example of issue on the Block Layout page:
Block layout alignment issue

An example of the issue on the Workflow configuration page:
Workflow alignment issue

An example of a good alignment on the Main Menu links page:
Menu links page

CommentFileSizeAuthor
#14 Screenshot 1943-02-17 at 10.41.33 AM.png273.58 KBSakthivel M
#12 Mozalla-seven.png92.05 KBdjsagar
#12 Chrome-seven.png86.59 KBdjsagar
#11 mozalla_seven.png83.79 KBvikashsoni
#11 chrome_seven_theme.png106.12 KBvikashsoni
#7 2867255-7.patch1.75 KBtar_inet
#6 2867255-6.patch1.81 KBtar_inet
#6 modernizr-touchevents.png66.05 KBtar_inet
#5 Firefox.54.0a2. Screen Shot 2017-04-10 at 2.40.58 PM.png252.13 KBJames.Mejia
#5 Chrome.57.0.2987.133.Screen Shot 2017-04-10 at 2.41.36 PM.png235.6 KBJames.Mejia
#4 Screen Shot 2017-04-07 at 9.59.31 AM.png22.4 KBashopin
#2 seven.jpg164.5 KBamit.mall
Screen Shot 2017-04-05 at 9.37.09 PM.png16.26 KBchris.smith
Screen Shot 2017-04-05 at 9.36.57 PM.png15.25 KBchris.smith
Screen Shot 2017-04-05 at 9.36.45 PM.png27.71 KBchris.smith
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chris.smith created an issue. See original summary.

amit.mall’s picture

amit.mall CreditAttribution: amit.mall at TO THE NEW commented
FileSize
seven.jpg164.5 KB

No such issue founded, I can see 10px vertical padding is already there.

seven

chris.smith’s picture

chris.smith CreditAttribution: chris.smith at Portage CyberTech commented

Thanks @amit.mall.

I tried to replicate the issue and was successful with only Google Chrome v57.

I was not able to replicate the issue with Safari or Firefox. The padding exists when using these browsers.

Would appreciate if someone else could help verify if the issue exists in Chrome 57.

ashopin’s picture

ashopin CreditAttribution: ashopin at Portage CyberTech commented
FileSize
Screen Shot 2017-04-07 at 9.59.31 AM.png22.4 KB

I also cannot replicate this issue in Acquia Dev Desktop + Chrome 57.0.2987.133 but can replicate it in Firefox 52.0.2 (64-bit).

James.Mejia’s picture

James.Mejia CreditAttribution: James.Mejia at Third and Grove commented
FileSize
Chrome.57.0.2987.133.Screen Shot 2017-04-10 at 2.41.36 PM.png235.6 KB
Firefox.54.0a2. Screen Shot 2017-04-10 at 2.40.58 PM.png252.13 KB

I wasn't able to replicate on Firefox 54 developer edition or Chrome 57. I'm on OSX.

Firefox 54.0a2 Developer Edition
Firefox 54
Chrome 57.0.2987
Chrome 57

tar_inet’s picture

tar_inet CreditAttribution: tar_inet as a volunteer commented
FileSize
modernizr-touchevents.png66.05 KB
2867255-6.patch1.81 KB

I can replicate it on Chrome 57.0.2987.133.

@chris.smith, do you have a touchable screen?

It looks like modernazr is adding the class "touchevents" to the html tag when using Chrome but not in Firefox.

Removing the space when on small device could make sense: reducing the vertical empty space you can see more rows. But I don´t really know why this is done by this way instead of using @media.

So my proposition to solve it is applying this styles only on smaller screens with no so much vertical space. I´ve search the stable styling guide but couldn´t find it so I don´t know if this a good way for doing it: I applied what I found similar in the stable´s css.

tar_inet’s picture

tar_inet CreditAttribution: tar_inet as a volunteer commented
Status: Active » Needs review
FileSize
2867255-7.patch1.75 KB

Sorry, wrong patch.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer commented

I have tried to reproduce this issue, but I can't seem to reproduce this in firefox or chrome on osx.

Version: 8.3.0 » 8.3.x-dev

Core issues are now filed against the dev versions where changes will be made. Document the specific release you are using in your issue comment. More information about choosing a version.

Version: 8.3.x-dev » 8.9.x-dev

Drupal 8.8.7 was released on June 3, 2020 and is the final full bugfix release for the Drupal 8.8.x series. Branches prior to 8.8.x are not supported, and Drupal 8.8.x will not receive any further development aside from security fixes. Sites should prepare to update to Drupal 8.9.0 or Drupal 9.0.0 for ongoing support.

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

vikashsoni’s picture

vikashsoni CreditAttribution: vikashsoni as a volunteer and at Zyxware Technologies commented
FileSize
chrome_seven_theme.png106.12 KB
mozalla_seven.png83.79 KB

No such issue founded, already proper alignment and padding there

djsagar’s picture

djsagar CreditAttribution: djsagar at OpenSense Labs commented
FileSize
Chrome-seven.png86.59 KB
Mozalla-seven.png92.05 KB

I have tried to reproduce this issue on chrome and mozalla but able to reproduce.

Please provide STR.

Thanks!

djsagar’s picture

djsagar CreditAttribution: djsagar at OpenSense Labs commented
Status: Needs review » Active
Sakthivel M’s picture

Sakthivel M CreditAttribution: Sakthivel M at QED42 commented
FileSize
Screenshot 1943-02-17 at 10.41.33 AM.png273.58 KB

I have tried to reproduce this issue on chrome but able to reproduce.

Attached screenshot here.

Thanks

Version: 8.9.x-dev » 9.2.x-dev

Drupal 8 is end-of-life as of November 17, 2021. There will not be further changes made to Drupal 8. Bugfixes are now made to the 9.3.x and higher branches only. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.2.x-dev » 9.3.x-dev

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.15 was released on June 1st, 2022 and is the final full bugfix release for the Drupal 9.3.x series. Drupal 9.3.x will not receive any further development aside from security fixes. Drupal 9 bug reports should be targeted for the 9.4.x-dev branch from now on, and new development or disruptive changes should be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Project: Drupal core » Seven
Version: 9.4.x-dev » 1.0.0-alpha1
Component: Seven theme » Code

The Seven theme has been removed from Drupal 10 core. It appears that this issue only affects Seven and no other themes included with Drupal core, so I am moving this to the contributed Seven project.