Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.I am using the PageTop->alter() function, but as I am on an admin page this function does not return anything.
It would be great to have a $theme variable on this function.
Issue fork ui_skins-3468192
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #3
rpayanmPlease review.
Comment #4
grimreaperHi,
I am ok with the change and the feature request.
Code quality tools needs to be fixed before merging.
Comment #5
grimreaperComment #6
grimreaperPlease give more details of what is your usage.
PageTop->alter() is just an OOP wrapper around hook before Core had the ability to declare hooks in classes with PHP attribute. So its signature should match the hook one. It is not intended to be reused anywhere in the code.
What can be done is part of PageTop->alter() could be extracted in a service or put directly into the UiSkinsUtility class.
Comment #7
grimreaper