- Issue created by @longwave
- ๐ฌ๐งUnited Kingdom longwave UK
While we're in this method the docblock is wrong as well:
/** * Overrides \Drupal\Core\Block\BlockBase::blockForm(). * * Adds body and description fields to the block configuration form. */
It doesn't do either of those things, it does allow the view mode to be set if more than one is available, but it probably should just be @inheritdoc.
- Assigned to prem suthar
- ๐ฎ๐ณIndia samit.310@gmail.com
samit.310@gmail.com โ made their first commit to this issueโs fork.
- Merge request !94743469736: Title field description override โ (Closed) created by samit.310@gmail.com
- Issue was unassigned.
- Status changed to Needs work
7 months ago 8:05am 12 September 2024 - ๐ฎ๐ณIndia samit.310@gmail.com
Hello,
I have moved
#description
tocore/lib/Drupal/Core/Block/BlockPluginTrait.php
as it seams generic.
Also changed the docblock ofblockForm
as it seams wrong.Thanks
Samit K. - Status changed to Needs review
7 months ago 9:55am 13 September 2024 - ๐ฌ๐งUnited Kingdom longwave UK
Personally I'm not sure the description really adds any value here, given it wasn't displayed before I don't think we should add it now.
- Status changed to Needs work
7 months ago 2:40pm 17 September 2024 - ๐ฎ๐ณIndia samit.310@gmail.com
@longwave, @smustgrave,
I guess it will be useful for new (inexperienced) developers. Description will help them what will be the use of that title field. A kind of help text.
Thanks
Samit K. - First commit to issue fork.
- ๐ฆ๐บAustralia acbramley
I've removed that added descriptions and rebased. Since this is such a trivial change I'm RTBCing. Thanks!
- ๐ฌ๐งUnited Kingdom catch
Also agreed on removing the description. Committed/pushed to 11.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.