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

BUG: fix go-select dropdowns being cut off in go-accordian-panels #267

Merged
merged 2 commits into from
Oct 14, 2019

Conversation

jaredami
Copy link
Contributor

@jaredami jaredami commented Oct 10, 2019

Closes #143

This PR addresses an issue in which GoSelect dropdowns are getting cutoff when their content expands below the bottom edge of a GoAccordionPanel. This was being caused by an overflow: hidden on the .go-accordion-panel css class. We can change this to overflow: visible, thus enabling the dropdown content to no longer get cut off, while also modifying the animation which occurs on expansion and collapse of a GoAccordionPanel to achieve the same effect as was provided by overflow: hidden previously.

@jaredami jaredami changed the title BUG: go accordian overflow BUG: fix go-select dropdowns being cut off in go-accordian-panels Oct 10, 2019
@jaredami jaredami changed the base branch from master to v1.3.0 October 10, 2019 14:54
@jaredami jaredami added the bug Something isn't working label Oct 10, 2019
@jaredami jaredami requested a review from grahamhency October 10, 2019 20:24
Copy link
Contributor

@grahamhency grahamhency left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes look good

@jaredami jaredami merged commit cb31eba into v1.3.0 Oct 14, 2019
@jaredami jaredami deleted the bug_go_accordian_overflow branch October 14, 2019 14:39
@grahamhency grahamhency added this to the v1.3.0 milestone Oct 14, 2019
@grahamhency grahamhency mentioned this pull request Oct 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants