Skip to content

Commit

Permalink
Issue #3486893, #3478484, #3483064, #3415662, by joshua1234511, richa…
Browse files Browse the repository at this point in the history
…rdgaunt, fionamorrison23, febdao, gargsuchi, sonam.chaturvedi, mwjansen, alex.skrypnyk: Add opt-in for controlling what form elements are rendered via civictheme system. (#1306)
  • Loading branch information
richardgaunt authored Nov 18, 2024
1 parent 5c9f5eb commit c4e9938
Show file tree
Hide file tree
Showing 12 changed files with 1,511 additions and 44 deletions.
383 changes: 383 additions & 0 deletions tests/behat/features/form_elements.core_form_elements.feature

Large diffs are not rendered by default.

265 changes: 265 additions & 0 deletions tests/behat/features/form_elements.webform_form_elements.feature

Large diffs are not rendered by default.

4 changes: 0 additions & 4 deletions web/modules/custom/civictheme_dev/civictheme_dev.module
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,6 @@ function civictheme_dev_preprocess_html(array &$variables): void {
* Implements hook_styleguide_alter().
*/
function civictheme_dev_styleguide_alter(array &$items): void {
// Remove password_confirm field from the form as CivicTheme does not support
// it.
unset($items['form']['content']['password_confirm']);

_civictheme_dev_styleguide__form_test__textfield($items);
_civictheme_dev_styleguide__form_test__textarea($items);
_civictheme_dev_styleguide__form_test__select($items);
Expand Down
14 changes: 14 additions & 0 deletions web/modules/custom/civictheme_dev/civictheme_dev.routing.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
civictheme_dev.styleguide_form:
path: '/civictheme-core-form-elements'
defaults:
_form: 'Drupal\civictheme_dev\Form\CoreFormElementsForm'
_title: 'CivicTheme Core Form Elements Form'
requirements:
_permission: 'access content'
civictheme_dev.admin_styleguide_form:
path: '/admin/structure/civictheme-core-form-elements'
defaults:
_form: 'Drupal\civictheme_dev\Form\CoreFormElementsForm'
_title: 'Core Form Elements Form (Admin Theme)'
requirements:
_permission: 'view the administration theme'
Loading

0 comments on commit c4e9938

Please sign in to comment.