Unclear term 'internal library name' in theme API documentation

I tried to add some clarity as to how libraries are referenced in the code as well as provide an example.

Seems like a good update.

Nice improvement!

It was still not immediately obvious to me how the library name was connected to the example when I first read the documentation out of context, so I suggested a slight improvement.

Thanks!

Saving issue credits.

I agree with #6, that is a bit clearer so I updated the MR. Thank you. Is that clear enough without having to state that the libraries are registered by $extension? I guess "Can either be 'core' or the machine name of the extension that registered the libraries." kinda clears that up.

CS failure (wish it would mark the issue NW):

FILE: /var/www/html/core/lib/Drupal/Core/Render/theme.api.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 1016 | ERROR | [x] Whitespace found at end of line
      |       |     (Squiz.WhiteSpace.SuperfluousWhitespace.EndLine)
----------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

I also think we should keep the first sentence about it being an associative array. It's not a library; it's an array of libraries. Right?

Gave it some more thought and came up with another revision.

Think this is good for committer review again.

lauriii → closed merge request !4754

Committed 01747e1 and pushed to 11.x. Also cherry-picked to 10.1.x as a docs fix. Thanks!

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