briancwald / material_admin

Drupal 8 Admin Theme
GNU General Public License v2.0
48 stars 15 forks source link

The "New" badge needs styles #4

Closed e0ipso closed 7 years ago

e0ipso commented 7 years ago

It needs (at least) some margin for readability.

content contenta json api 2017-05-20 12-13-05

briancwald commented 7 years ago

@e0ipso with vanilla drupal my content page looks like this:

screenshot 2017-05-20 12 58 44

Are you modifying that view somehow to remove the bulk operations? I'm not sure why I am not getting 'new' tag either.

briancwald commented 7 years ago

Got it, you start with minimal profile build which doesnt have any of the bulk operation features

screenshot 2017-05-20 13 08 38

I have never built this profile as a starting point using Material Admin yet, I will poke around (and fix this issue).

briancwald commented 7 years ago

fixed drupal 'marker' span class to use badges css component.

screenshot 2017-05-20 13 26 20
e0ipso commented 7 years ago

Wow! You're killing it. Thanks.

On May 20, 2017 19:29, "Brian" notifications@github.com wrote:

fixed drupal 'marker' span class to use badges css component. [image: screenshot 2017-05-20 13 26 20] https://cloud.githubusercontent.com/assets/776942/26278023/5a2c3a9a-3d60-11e7-9f0e-0ff7bf9bfcc5.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/briancwald/material_admin/issues/4#issuecomment-302887231, or mute the thread https://github.com/notifications/unsubscribe-auth/ABFoqoZGmccXHHPtlCdImFcSVupYisQFks5r7yMCgaJpZM4NhQUj .