When bartik is configured to be the theme for node creation/editing the image upload widget in bartik has no border around it.

image widget

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Bug as the file field does not have any CSS styling in Bartik, other field form elements on content add/edit pages do.
Unfrozen changes Unfrozen because it only changes CSS.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

emma.maria’s picture

Version: 8.0.0-beta2 » 8.0.x-dev
Issue tags: -#Drupal8 #Theme +frontend, +CSS, +Novice
FileSize
49.93 KB

I can confirm there is no border on image file uploads in Bartik.

Seven has a border like this on image file uploads.

I think we should replicate this in Bartik to tidy it up. There are styles for borders on inputs already in place, we just need to add the selector for file-uploads to this.

vermario’s picture

Assigned: Unassigned » vermario

Working on this! :)

emma.maria’s picture

Thanks @vermario, looking forward to seeing the results.
If you get stuck with something don't worry, just post what you have and we will all help you out :)

vermario’s picture

Status: Needs work » Needs review
FileSize
65.1 KB
378 bytes

Here is the one-liner patch. :-)

Screenshot:

emma.maria’s picture

Assigned: vermario » Unassigned
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
FileSize
85.68 KB
31.24 KB

@vermario For your next patch can you look at the patch naming conventions https://www.drupal.org/patch/submit/#patch_naming. People sometimes swap the issue number and description around but it's important that all those items are in the patch name.
Also don't forget to un-assign from an issue when you set to Needs Review (I forget all the time) this is just incase people ignore the issue as they think you are reviewing it.
Thanks for working on this Mario, I've heard you are doing really well getting started with Core and it shows :)

Patch review:

The image file field now has a border around it and looks good, so much cleaner. The selector used also matches the general file field so both now look great. Win win.

Only local images are allowed.

I also reviewed the code and the CSS selector was added along with other fields that use the border style so no replication of CSS styles, so the code is all good too.

Setting to RTBC.

emma.maria’s picture

Issue summary: View changes
emma.maria’s picture

Issue summary: View changes
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed e89029c and pushed to 8.0.x. Thanks!

Thanks for adding the beta evaluation for to the issue summary.

  • alexpott committed e89029c on 8.0.x
    Issue #2368251 by emma.maria, vermario, tkoleary: No border around image...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.