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

CommentFileSizeAuthor
#14 bartik_node_form_7-1000688-14.patch443 bytesKeyboardCowboy
#13 1000688-node-form-css.png52.12 KBKeyboardCowboy
#12 1000688-bartik_node_form-8.patch666 bytesfranz
#8 1000688-bartik_node_form-8.patch685 bytesAnonymous (not verified)
#5 1000688-bartik_node_form-5.patch659 bytesAnonymous (not verified)
#2 1000688-bartik_node_form.patch639 bytesfranz

Comments

jensimmons’s picture

jensimmons commented

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

franz’s picture

franz
any
Location Montréal
commented
Version: 7.x-dev » 8.x-dev
Status: Active » Needs review
StatusFileSize
new 1000688-bartik_node_form.patch639 bytes

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

franz’s picture

franz
any
Location Montréal
commented
Issue tags: +Needs backport to D7

Adding tag

franz’s picture

franz
any
Location Montréal
commented
Status: Needs review » Needs work

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

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new 1000688-bartik_node_form-5.patch659 bytes

Rerolled for new D8 directory structure (/core).

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Needs review
franz’s picture

franz
any
Location Montréal
commented
Status: Needs review » Needs work

@ArtistConk, please see my comment on #4

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Needs review
StatusFileSize
new 1000688-bartik_node_form-8.patch685 bytes

New patch with updated font for "Tags" description.

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
commented

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

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
commented
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

franz
any
Location Montréal
commented
Status: Needs work » Needs review
StatusFileSize
new 1000688-bartik_node_form-8.patch666 bytes

Re-rolled

KeyboardCowboy’s picture

KeyboardCowboy
Primary language English
Location Denver, CO, USA
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new 1000688-node-form-css.png52.12 KB

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

KeyboardCowboy’s picture

KeyboardCowboy
Primary language English
Location Denver, CO, USA
commented
StatusFileSize
new bartik_node_form_7-1000688-14.patch443 bytes

Adding patch for D7.

KeyboardCowboy’s picture

KeyboardCowboy
Primary language English
Location Denver, CO, USA
commented
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

franz
any
Location Montréal
commented
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

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Issue tags: -Needs backport to D7

#14: bartik_node_form_7-1000688-14.patch queued for re-testing.

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

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs work » Reviewed & tested by the community

Oops so #14 is the D7 patch

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
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

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Patch (to be ported) » Needs review
Issue tags: -Needs backport to D7

#14: bartik_node_form_7-1000688-14.patch queued for re-testing.

dcam queued 14: bartik_node_form_7-1000688-14.patch for re-testing.

  • 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...

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.