Add support for the form display

Created on 28 March 2024, 8 months ago
Updated 12 September 2024, 2 months ago

Problem/Motivation

I'd like to add custom classes on the node creation form on Drupal 10, for example on admin/structure/types/manage/[my_node_type]/form-display, the module only works for the field output.

Is there a way to extend your module even for input forms?

โœจ Feature request
Status

Fixed

Version

1.0

Component

Code

Created by

๐Ÿ‡ฎ๐Ÿ‡นItaly smurfxx

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @smurfxx
  • ๐Ÿ‡ฆ๐Ÿ‡บAustralia Nigel Cunningham Geelong

    Nigel Cunningham โ†’ made their first commit to this issueโ€™s fork.

  • Status changed to Needs review 5 months ago
  • ๐Ÿ‡ฆ๐Ÿ‡บAustralia Nigel Cunningham Geelong

    Here's an initial version, tested on a site I'm building with nodes and eck entities so far.

  • ๐Ÿ‡ซ๐Ÿ‡ทFrance dqd London | N.Y.C | Paris | Hamburg | Berlin

    The attempt looks good to me. Did you experienced any drawbacks yet in the time of using? Did this project uses tests? If so we may need to add tests for this.

  • ๐Ÿ‡ซ๐Ÿ‡ทFrance dqd London | N.Y.C | Paris | Hamburg | Berlin

    Also, we should try to add this to latest dev and possibly also to latest release which is 8.x 1.7.

  • Status changed to Needs work 3 months ago
  • ๐Ÿ‡ซ๐Ÿ‡ทFrance dqd London | N.Y.C | Paris | Hamburg | Berlin

    Accordingly #5 this needs a re-roll...

  • Merge request !5Add support for the form display. โ†’ (Merged) created by dqd
  • Status changed to RTBC 3 months ago
  • ๐Ÿ‡ซ๐Ÿ‡ทFrance dqd London | N.Y.C | Paris | Hamburg | Berlin

    Good news.

    $ wget https://git.drupalcode.org/project/field_formatter_class/-/merge_requests/5.patch
    --2024-08-28 17:03:24--  https://git.drupalcode.org/project/field_formatter_class/-/merge_requests/5.patch
    Resolving git.drupalcode.org (git.drupalcode.org)... 146.75.122.217
    Connecting to git.drupalcode.org (git.drupalcode.org)|146.75.122.217|:443... connected.
    HTTP request sent, awaiting response... 200 OK
    Length: unspecified [text/plain]
    Saving to: '5.patch'
    5.patch      [ <=>  ]   4.47K  --.-KB/s    in 0s
    2024-08-28 17:03:24 (17.0 MB/s) - '5.patch' saved [4581]
    
    $ git apply -v 5.patch
    Checking patch field_formatter_class.module...
    Hunk #2 succeeded at 99 (offset 1 line).
    Applied patch field_formatter_class.module cleanly.
    

    Ahead but still applies and works, so ... if one of the maintainers is quick enough, then ... ;-)

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States Chris Matthews

    chris matthews โ†’ made their first commit to this issueโ€™s fork.

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States Chris Matthews
  • Status changed to Fixed 3 months ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States Chris Matthews

    Fixed in 8.x-1.8.
    Thanks!

  • ๐Ÿ‡ซ๐Ÿ‡ทFrance dqd London | N.Y.C | Paris | Hamburg | Berlin

    Great reaction @ChrisMatthews, so it's merged to latest dev now. But let us keep issue titles and commit/merge comments in sync.

  • ๐Ÿ‡ซ๐Ÿ‡ทFrance dqd London | N.Y.C | Paris | Hamburg | Berlin

    Oops, sorry I didn't had your latest comment in my open tab from last night visible yet (only the commits) and I wanted to close the issue now. But you did already :) hah. And yeah, it's an FR indeed.

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

Production build 0.71.5 2024