Quantcast
Channel: Issues for Drupal core
Viewing all 292571 articles
Browse latest View live

Hover effect while creating image style.

$
0
0

When creating the image style and hover on the anchor instead of hovering over the particular box it is showing a hover effect on the whole row.
Steps to Reproduce

  1. Create new image style
  2. Add Scale and Crop as a style

For example Hovering on A11 Brings hover effect on A12 and A13 also


When you remove the "Breadcrumb" block in Seven, there's no top padding for the main content

$
0
0

Problem/Motivation

With

Without

Proposed resolution

Fix CSS.

Remaining tasks

Browser tests

User interface changes

minimal, better spacing when breadcrumb block is disabled on admin pages

API changes

none

Data model changes

none

Migrate sets revision_translation_affected incorrectly when different languages co-exist for 8.x un-translatable fields

Fix flagged spelling errors due to missing hyphens for prefixes

$
0
0

Problem/Motivation

Discovered at #2972224: Add .cspell.json to automate spellchecking in Drupal core, and pointed by @xjm in https://www.drupal.org/project/drupal/issues/3122088#comment-13628724

+++ b/core/.cspell.json
@@ -0,0 +1,1288 @@
+      "Deduplicates",
...
+      "Reimplement",
...
+      "Reparenting",
...
+      "Rerender",
+      "Resaving",
+      "Rethrown",
+      "Reuploading",
...
+      "Unassigns",
...
+      "Unclickable",
+      "Ungroupable",
+      "Ungrouped",
+      "Unhide",
...
+      "Unknow",
...
+      "Unpad",
...
+      "Unprefix",
...
+      "Unrouted",
...
+      "deprioritize",
...
+      "reimplementing",
+      "reindex",
+      "reindexing",
+      "reinitializes",
+      "reinject",
...
+      "renormalize",
+      "reparenting",
...
+      "rerender",
+      "rerendered",
+      "rerendering",
+      "resaved",
+      "resaving",
...
+      "rethrown",
...
+      "undecorate",
...
+      "ungroup",
+      "ungroupable",
+      "ungrouped",
+      "unhashed",
+      "unhide",
+      "unindented",
+      "unindexed",
...
+      "unitalicize",
+      "unkeyed",
...
+      "unpermissioned",
...
+      "unrendered",
...
+      "unshortened",
+      "unsanitized"
+
@xjm:

Should be hyphenated (un-assign, de-prioritize, re-render, etc.)

There's an interesting question of where to draw the line for these. Generally in English these prefixes are morphologically productive with a hyphen, and get de-hyphenated when the word is adopted into common usage. "Denormalize", "Unsanitized", "Uninstantiated" etc. are all obviously in common usage in programming. "Unsticky", "Unrevisionable", and the like are Drupal terminology, and I'm surprised that "unpublish" and friends aren't already in the main dictionary.

Proposed resolution

As title and see the change record https://www.drupal.org/node/3122084 for how to work with cspell.

Remaining tasks

Pick out all applicable words from #2972224: Add .cspell.json to automate spellchecking in Drupal core and fix them.

User interface changes

API changes

Data model changes

Release notes snippet

Views Exposed Filter Auto Complete "Has taxonomy term" field too wide

$
0
0

In a fresh install of Drupal 8.8.5, when adding an exposed filter using the criteria of "Has taxonomy term", the field is twice as wide as all other fields that are exposed. I have attached images. The default character length in most text based fields is 30 but in "has taxonomy term" it is set at 60. This makes the filter region look like ... you know.

I know I have to find a way to make a sub theme just to fix that one little item, but the more I thought about it is a minor bug and one that is easily fixed in Core.

See attached files

Remove focus effect from non-interactive elements in Internet Explorer 11

$
0
0

Problem/Motivation

To reproduce this issue, view the html in #9 in IE11.

In IE11, any element set to display: flex; can receive focus by being clicked. This has been narrowed down to an IE bug, but one that has very little evidence of it existing online because it is only noticeable if stylesheets include rules that provide focus outlines to non-interactive elements. This is the case with Claro's :focus styles, as they are are applied as *:focus, impacting all elements. This can result in the green focus ring appearing in unexpected places.

The bug does not result in making these elements tabbable via tab navigation, but clicking on one of these non-interactive-but-focusable elements will take focus away from another element .

Proposed resolution

Any number of CSS rules could address the problem.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Resolve @todo: Remove this part once we have a _title_callback, see https://www.drupal.org/node/2076085.

Add an implementation of EntityDisplayBase::label()

$
0
0

Problem/Motivation

Neither EntityFormDisplay nor EntityViewDisplay have a label key, and as such their ::label() method returns NULL.

Proposed resolution

Implement ::label(), returning a string indicating which entity type, bundle, and view mode are represented.

Remaining tasks

Decide on the string to present and then use it in the Field UI.

User interface changes

Steps to check the label with the Layout builder module installed first:

Node:【Structure】->【Content types】->【Article】->【Manage Display】->【Default】/【Teaser】mode ->【 Use Layout Builder】->【Manage Layout】

User:【Configuration】->【Account Settings】->【Manage Display】->【Default】mode ->【 Use Layout Builder】->【Manage Layout】

API changes

Data model changes


Element inside form-item can cause overflow

$
0
0

Elements inside a form-item can cause unwanted overflow to happen (see screenshot).
Path: admin/content
Patch follows.

Remove @todo : Remove after 8.6.x is out of support in function claro_preprocess_table

Improve StringItem::generateSampleValue()

$
0
0

Problem/Motivation

Sample entity value generation generates long strings for text fields. This can cause the layout to break, as noted in #3016507: Break long text strings in layout edit

Proposed resolution

Rather than use \Drupal\Component\Utility\Random::word, which often generates very long words, use \Drupal\Component\Utility\Random::paragraphs, which uses words from a selection of shorter words.

Remaining tasks

Write a patch.

User interface changes

None-ish.

API changes

None

Data model changes

None

[META] Provide modern replacements for and deprecate the legacy include files before Drupal 10

$
0
0

Drupal 9 still has a bunch of legacies includes files. See the previous meta #2999721: [META] Deprecate the legacy include files before Drupal 9

Here is the tracking of the deprecation status of the legacy include files in the core modules and core/includes directory:

Includes files

Bootstrap

Kernel includes

Install includes

Rest of the files

Modules

Let the 'Transform dashes in URL to spaces in term name filter values' find terms that should contain the dash

$
0
0

Problem/Motivation

Follow up to #2710407: Option for 'Transform dashes in URL to spaces in term name filter values' on term arguments doesn't affect the query

If you have a term "co-worker" and you enable 'Transform dashes in URL to spaces in term name filter values', no matches will be found because the dash will be removed.

So we need to figure out how to handle the case where you have both terms with spaces and terms with dashes, since now it only works with one or the other.

Also need to figure out how to handle this when you have both "co worker" and "co-worker" as terms.

Proposed resolution

If the 'Transform dashes in URL to spaces in term name filter values' checkbox is set, search for terms that match either a space or a dash for any dashes in the argument value.

If multiple terms match, use the most recently created term.

Remaining tasks

  1. UX review of '#description' => $this->t('You can have both terms with spaces and terms with dashes like "co worker" and "co-worker". Then only the last created term will be matched.'),
    • Is that the behavior we want? Should it be "last" term, "first" term, other?
    • Is that description text good?
  2. Other reviews / refinements.
  3. RTBC.
  4. Commit.

User interface changes

Adds this description to the 'Transform dashes in URL to spaces in term name filter values' checkbox on the TermName views argument validator options form:

You can have both terms with spaces and terms with dashes like "co worker" and "co-worker". Then only the last created term will be matched.

API changes

None.

Data model changes

None.

Release notes snippet

TBD.

Add back rollbacks through the UI

$
0
0

Follow-up to #2687843: Add back incremental migrations through the UI

Problem/Motivation

Follow-up to #2683421: Remove incremental and rollback options from the UI (and add them back when they are more stable) which removes both the rollback and incremental options since they were untested and had several significant bugs. Unlike the incremental migration feature, rollback can probably be considered an additional feature and not necessarily block a stable Migrate UI.

Proposed resolution

Add back rollback migration support to the UI, with test coverage.

Use the phrase 'incremental upgrade' instead of 'Rerun' as shown in the now dated screenshot. See #32

Documentation to the handbook will be covered in a follow-up issue #2947498: Document Migrate Drupal UI rollbacks.

Remaining tasks

Documentation to the handbook will be covered in a follow-up issue #2947498: Document Migrate Drupal UI rollbacks.

User interface changes

In #2683421: Remove incremental and rollback options from the UI (and add them back when they are more stable), the UI disallows both incremental migrations and rollbacks. Here is what it looked like before that:

We might want to either reimplement that, or change it.

API changes

Depends on the solution.

Data model changes

None

Postponed until

Resolve @todo: when https://www.drupal.org/project/drupal/issues/2999549 * lands.


Rename dictionary.txt file to drupal.dic to make it compatible with PhpStorm

$
0
0

Problem/Motivation

Since #2972224: Add .cspell.json to automate spellchecking in Drupal core Drupal core ships with a spell check dictionary stored as plain text file (dictionary.txt). That file could be used as custom dictionary in PhpStorm as well. Unfortunately PhpStorm only accepts files with "dic" extension.
https://www.jetbrains.com/help/phpstorm/spellchecking.html#configure-the...

Proposed resolution

Given that cspell can accept any file extensions we can configure it to use "dic" file instead of "txt". That will make possible to use the same dictionary file in PhpStorm and cspell.

Replace jQuery UI dialog with supported library or polyfill

$
0
0

Problem/Motivation

We are in the process of deprecating jQuery UI in core. jQueryUI dialog is among the components that need to be removed/replaced.
As mentioned in the parent issue: #3067261: [Plan] Remove jQuery UI components used by Drupal core and replace with a set of supported solutions
The OpenJS Foundation lists jQuery UI as an Emeritus project in https://openjsf.org/projects/ which is described as:

Emeritus projects are those which the maintainers feel have reached or are nearing end-of-life

This issue was originally proposed as specifically looking for a polyfill, with the following explanation:

dialog.js always aimed at using the HTML5 dialog spec. Chrome just added experimental support for dialogs in it's dev branch: http://demo.agektmr.com/dialog/

We should let chrome use native dialogs when needed, it actually solves a bunch of messy issues, two of which are:
#2072037: Drupal dialog modal background z-index is set too low to reliably occlude core UI components
#1836392: In the Views UI, the interaction pattern of “All displays”/ “Override this display” is confusing

The problems mentioned above should absolutely be considered wile investigating what to use in place of jQueryUI dialog, but the options should be expanded to include different libraries - whatever option can best facilitate removing jQuery UI.

Proposed resolution

Refactor dialog JS to allow use of Native dialogs when available, and do so in a way that provides a consistent user experience regardless of browser. This will double as an effective way to swap libraries in the future, as it requires removing the library-specific assumptions of the current implementation.

Implement the replacement(s) so the dialogs have the same structure/classes as the current jQueryUI implementation. This will allow core to continue using jQueryUI's CSS styles and reduce the overall number of changes that result from this change (desired changes to the styling/structure of dialogs can obviously be done later, it just won't happen here to keep the issue's already-huge scope from getting huger.

Remaining tasks

Refactor JS to provide conditional use of native dialogs -- this implementation should hopefully make it relatively easy to add the replacement library once one is found.

Find a library that meets these requirements

  1. Able to open/close dialogs via Javascript. It can't rely on being coupled with buttons or other UI elements.
  2. Supports non-modal dialogs (i.e. there has to be the option for the entire page to be interactable even when the dialog is open)
  3. Multiple dialogs can be open at the same time, for situations such as adding a custom block in Layout Builder and triggering the media library embed modal via WYSIWYG. They should be able to stack in the order they were opened.

(Chrome native dialogs have been tested and confirmed as meeting these requirements)

The best candidate is currently https://github.com/GoogleChrome/dialog-polyfill -- this meets all the requirements, is actively supported and initial tests/prototypes are promising.

If we are fortunate enough to find multiple libraries that meet these requirements, they should be compared using criteria such as: Code Style, Maturity, Responsiveness, Accessibility, UX, UI, Modularity, etc.

Libraries reviewed so far:

.

Libraries to review

Implement the library

Add additional tests for important dialog use cases that aren't part of existing coverage. Before doing this review Layout Builder's test coverage as it indirectly tests a wide variety of Dialog use cases.

User interface changes

Yes, TBD.

API changes

TBD.

Data model changes

Release notes snippet

Attached is the patched used to make the following work (minus the scrolling element, that's hardcoded CSS).

What needs to be done is making sure our API can handle native dialogs when available and that pretty much means we need to not use jQuery UI or in a very different way than today to avoid wildly different UX between native and polyfill.

[PP-2] Find a way to notify people about Symfony Event class changes

$
0
0

Problem/Motivation

Following #3055198: [Symfony 5] Symfony/Component/EventDispatcher/Event is deprecated in Symfony 4.3 use Symfony/Contracts/EventDispatcher/Event instead and #3055194: [Symfony 5] The signature of the "Drupal\Component\EventDispatcher\ContainerAwareEventDispatcher::dispatch()" method should be updated to "dispatch($event, string $eventName = null)", not doing so is deprecated since Symfony 4.3. Drupal core will no longer have an explicit dependency on the Symfony EventDispatcher Event class which was deprecated in Symfony 4 and replaced by an identical class in the Symfony/Contracts namespace.

However, Symfony Event's own classes (like GenericEvent) still inherit from the deprecated class, and some of Drupal's classes like EntityTypeEvent inherit from GenericEvent.

Because GenericEvent is going to stay in EventDispatcher in Symfony 5 (with the use statement updated), there is no need for any classes inheriting from GenericEvent to do anything - they'll get 'auto-updated' when we update to Symfony 5.

However, this presents a twofold problem:

1. We are currently suppressing Symfony's deprecation message, and there is no way to un-suppress this while the Symfony/EventDispatcher version is in use.

2. This means we're not able to inform people about the bridge Event class added in #3055198: [Symfony 5] Symfony/Component/EventDispatcher/Event is deprecated in Symfony 4.3 use Symfony/Contracts/EventDispatcher/Event instead until at least both of those issues are complete.

Proposed resolution

It should be possible to add a drupal check/rector rule looking for explicit references to the deprecated Event class in use statements.

We might be able to check the inheritance chain in Drupal's ContainerAwareEventDispatcher::dispatch(), and for example trigger deprecation errors for classes that directly inherit from the deprecated class - however this will also depend on Symfony's own Event classes.

Another option is to replace classes via the classloader so that we can customize the deprecation messages.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Use dropbutton variants with #dropbutton_type instead of custom classes

Image style is not working after upgrading to drupal 7.72

$
0
0

Hi all,

Could someone pls help me to fix this.

The image style module was creating the images before upgrading to drupal 7.72. The thumbnail images and other image style images are or not created after upgrading to drupal 7.72 version. It started working after downgrading to drupal 7.70 version.

Any help is much appreciated

Regards,
Suganya M

Viewing all 292571 articles
Browse latest View live


<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>