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

Fix ghost links in nested tabs #500

Merged
merged 2 commits into from
May 15, 2023
Merged

Conversation

Splines
Copy link
Member

@Splines Splines commented May 11, 2023

image

While viewing the media of a course, links were randomly placed and invisible (e.g. on the route: http://localhost:3000/courses/2/edit). This was (mainly) due to a wrongly placed bootstrap class tab-content inside a column layout (wrapping only one column instead of the whole row).

@Splines Splines added the bug label May 11, 2023
@Splines Splines requested a review from fosterfarrell9 May 11, 2023 16:46
@Splines Splines self-assigned this May 11, 2023
@codecov
Copy link

codecov bot commented May 11, 2023

Codecov Report

Merging #500 (ee5c1a1) into mampf-next (5f7c956) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           mampf-next     #500   +/-   ##
===========================================
  Coverage       66.56%   66.56%           
===========================================
  Files             311      311           
  Lines            9376     9376           
===========================================
  Hits             6241     6241           
  Misses           3135     3135           

@Splines
Copy link
Member Author

Splines commented May 11, 2023

still have to fix a small issue because I removed the margin-left

image

@Splines
Copy link
Member Author

Splines commented May 15, 2023

no overlap anymore ;)
image

@Splines Splines merged commit 3fc077f into mampf-next May 15, 2023
@Splines Splines deleted the fix/links-in-media-card-partial branch May 15, 2023 12:59
@Splines Splines mentioned this pull request May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants