Many times, people configure a website so that the fonrt-facing theme is used for adding content instead of the admin theme. So the node add form for Bartik needs to look good. It needs some visual help.

screenshot of Bartik's node add form

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jensimmons’s picture

I think there's a bit too much top padding on the 'panels' (items?) for the vertical tabs, too.

franz’s picture

Version: 7.x-dev » 8.x-dev
Status: Active » Needs review
FileSize
639 bytes

I did some CSS changes to fix those issues. I don't thing the padding inside VT is too much.

franz’s picture

Issue tags: +Needs backport to D7

Adding tag

franz’s picture

Status: Needs review » Needs work

Ah, I guess the "Tags" description is still with a wrong font

Anonymous’s picture

Rerolled for new D8 directory structure (/core).

Anonymous’s picture

Status: Needs work » Needs review
franz’s picture

Status: Needs review » Needs work

@ArtistConk, please see my comment on #4

Anonymous’s picture

Status: Needs work » Needs review
FileSize
685 bytes

New patch with updated font for "Tags" description.

kscheirer’s picture

Retesting against latest HEAD since it has been over a year.

kscheirer’s picture

Issue tags: -Needs backport to D7

#8: 1000688-bartik_node_form-8.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Needs backport to D7

The last submitted patch, 1000688-bartik_node_form-8.patch, failed testing.

franz’s picture

Status: Needs work » Needs review
FileSize
666 bytes

Re-rolled

KeyboardCowboy’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
52.12 KB

Patch in #12 works as designed. Increased padding around image field. Correct font on node form labels.

KeyboardCowboy’s picture

Adding patch for D7.

KeyboardCowboy’s picture

Status: Reviewed & tested by the community » Patch (to be ported)

Changing status.

As of this comment, D8 patch has been tested and works. D7 patch needs to be ported and tested.

franz’s picture

Status: Patch (to be ported) » Reviewed & tested by the community

Thanks KeyboardCowboy, but leave it RTBC until it's committed to D8, otherwise it'll fall off the radar.

alexpott’s picture

Issue tags: -Needs backport to D7

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs backport to D7

The last submitted patch, bartik_node_form_7-1000688-14.patch, failed testing.

alexpott’s picture

Status: Needs work » Reviewed & tested by the community

Oops so #14 is the D7 patch

alexpott’s picture

Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed 4f3fbf6 and pushed to 8.x. Thanks!

alexpott’s picture

Status: Patch (to be ported) » Needs review
Issue tags: -Needs backport to D7

  • alexpott committed 4f3fbf6 on 8.3.x
    Issue #1000688 by ArtistConk, franz: Fixed Bartik's node add form needs...

  • alexpott committed 4f3fbf6 on 8.3.x
    Issue #1000688 by ArtistConk, franz: Fixed Bartik's node add form needs...

  • alexpott committed 4f3fbf6 on 8.4.x
    Issue #1000688 by ArtistConk, franz: Fixed Bartik's node add form needs...

  • alexpott committed 4f3fbf6 on 8.4.x
    Issue #1000688 by ArtistConk, franz: Fixed Bartik's node add form needs...