[PP-2] Add the content moderation state widget to the layout builder entity...
Problem/MotivationBlocked on:#3004536: Move the Layout Builder UI into an entity form for better integration with other content authoring modules and core features#2942907: Entity system does not...
View ArticleImprove test coverage for node authored on widget
Problem/MotivationAdd test coverage for creation date. Make sure that there are no errors when the user create a node with a different "Authored on" than the default or the date or time is...
View ArticleConditionally change the query operator in between exposed filter when empty...
Problem/MotivationWhen exposing between filters, if a user only enters one value (min or max), the query that is run always returns odd results (most likely db-backend-dependent. For instance, on an...
View ArticleChanging RTL styling guidelines
Problem/MotivationThe current CSS guidelines for Drupal 8 #1887862 requires adding inline /* LTR */ comments that make the code difficult to read, and requires redefining LTR rules when writing RTL...
View ArticleLIST(Text) field HTML 5 Validation
I have created the field (LIST - Text) in content type basic page (See screenshot) and make it required with no default value. Then, going to add new page it is validating title via HTML 5 validation...
View ArticleNo red outline around invalid slider range
There is an error with invalid form submissions. On Chrome 71.0.3578.98 on Mac OSX 10.14.2, Safari 12.0.2 and Opera 58.0.3135.47 there is no red outline around the slider range when an invalid range is...
View ArticleClik here to view.
States API doesn't work with multiple select fields
The States API won't pick up the value of a select field with #multiple = TRUE option.Steps to reproduce: - Create two fields in a form with the following code: $form['dependee'] = array( '#type' =>...
View ArticleContent moderation show state label instead of transition label
Problem/MotivationWhen attaching a content moderation workflow to a content, "change to" select list show state label instead of showing transition label.Proposed resolutionShow transition label...
View ArticleText fails contrast in banner
Banner Area Fails Color Contrast TestingThe banner on the front page fails contrast testing. Testing was conducted using the Color Contrast Analyzer plugin for Chrome.Part of this is due to the...
View ArticleMigrate taxonomy term references for D6 Node translations
Problem/MotivationIn Drupal 6 Node translation concept each language version is a separate node with their own nid. Each of these nodes can have their own taxonomy term references.In Drupal 6, you can...
View ArticleFix for aliasExists which fails if langcode is empty
I also found an issue while creating new content, it did not check the existing aliases if langcode was set to null and was able to fix it by adding an if condition on $langcodeif ($langcode){...
View ArticleMigrate D6 and D7 node revision translations to D8
Postponed on #2975509: Migrate D6 vocabulary language settings and #2886609: Migrate translations for D6 i18n taxonomy 'localized' termsFollow-up to #2225775: Migrate Drupal 6 core node translation to...
View ArticleUmami - hero block is identical on Home and Recipe pages, so potentially...
Problem/MotivationThe hero block, used on the Home and Recipes pages, is identical across pages and causes confusion. It also could use better text for the button.Proposed resolutionAdd a new hero...
View Article"Remember" feature of exposed filters doesn't work with AJAX...
Remembered filter values are restored in view::get_exposed_input() function only when there is no input from GET parameters. However, AJAX view rendering always adds lots of parameters, so input is...
View ArticleUnnecessary <strong> element in Classy's form-element template may...
Problem/MotivationThe form-element.html.twig template in Classy wraps inline error messages in a <strong> element.The minor issue is that it makes no sense to denote the whole error message.The...
View ArticleRemove use of MigrateCckFieldInterface in FieldDiscovery service
The FieldDiscovery service includes usage of MigrateCckFieldInterface and this needs to be remove for Drupal 9. Here is the code, from patch #103 in #2951550: Make service for field discovery for use...
View Articlehook_node_submit lists wrong invocation
API page: https://api.drupal.org/api/drupal/modules%21node%21node.api.php/function...The invocation of this hook happens in node_form_submit_build_node(), but the Invocation section only lists...
View Articledrupal_deliver_html_page() might call drupal_render_page() with an integer...
SummaryIn drupal_deliver_html_page() in some circumstances menu_execute_active_handler() is called and its return value is directly passed to drupal_render_page() even if it is an integerSteps to...
View ArticleConvert AJAX part of \Drupal\file\Tests\FileFieldWidgetTest to...
Problem/MotivationdrupalPostAjaxForm() is simulating the behaviour of ajax.js, so using it, doesn't really provide fundamental guarantees.#2809161: Convert Javascript/AJAX testing to use...
View ArticleNotice: Undefined index: #type in drupal_process_states()
Problem/MotivationProblemWhen #states is attached to a form element that contains #markup the following notice occurs:Notice: Undefined index: #type in drupal_process_states() (line 583 of...
View Article