Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Radio buttons are not styled when showing forms in modals #26775

Merged
merged 7 commits into from Jan 9, 2020

Conversation

obuisard
Copy link
Contributor

@obuisard obuisard commented Oct 22, 2019

Pull Request for Issue #26774.

Summary of Changes

Added template.js to component.php in Protostar.

Testing Instructions

Open a form containing radio buttons inside a modal while using Protostar.
(Sorry, I have noticed while testing a commercial extension, therefore I have no example to provide).

What you could try:

  • Create a contact mail custom field of radio type.

  • Create a menu item of type 'List contacts in a category'.

  • Create an override of the com_contacts component (category view).

  • Modify the link to the contact items in the default_items.php override to force the modals (just for test purposes):

JRoute::_(ContactHelperRoute::getContactRoute($item->slug, $item->catid)).'&tmpl=component'
In the frontend, this will open the contacts into component.php rather than index.php.

Expected result

The radio buttons are styled.

Actual result

The radio buttons are not styled.

Documentation Changes Required

None.

obuisard and others added 7 commits February 17, 2018 14:10
Duplicate entry .radio.btn-group > label:first-of-type
Removed 2nd occurrence of  .btn-group > .btn:first-child, .radio.btn-group > label:first-of-type
When showing forms in modals (for instance), radio buttons are not skinned and are actually partially hidden (because CSS from template.css hides the clickable buttons, only showing the labels).
This is the result of the missing template.js, which adds the proper class names to the form elements.
Missing template.js in component.php of Protostar
Refresh of original Joomla repository
@SharkyKZ
Copy link
Contributor

SharkyKZ commented Dec 9, 2019

I have tested this item ✅ successfully on 8abae6e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26775.

1 similar comment
@jwaisner
Copy link
Member

jwaisner commented Jan 8, 2020

I have tested this item ✅ successfully on 8abae6e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26775.

@SharkyKZ
Copy link
Contributor

SharkyKZ commented Jan 8, 2020

RTC.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26775.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 8, 2020
@HLeithner
Copy link
Member

Thanks

@HLeithner HLeithner merged commit 31b76ff into joomla:staging Jan 9, 2020
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 9, 2020
@HLeithner HLeithner added this to the Joomla! 3.9.15 milestone Jan 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants