Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Support
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
Minds Frontend
Minds Frontend
  • Project
    • Project
    • Details
    • Activity
    • Releases
    • Cycle Analytics
    • Insights
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 868
    • Issues 868
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 54
    • Merge Requests 54
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Packages
    • Packages
    • List
    • Container Registry
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds FrontendMinds Frontend
  • Merge Requests
  • !467

Merged
Opened 3 weeks ago by Olivia Madrid@omadrid22 of 22 tasks completed22/22 tasks
  • Report abuse
Report abuse

[Sprint/KiltedKoala](feat): Media modal

To do:

  • Click video thumbnail to open modal and autoplay
  • Get dimensions of video before modal dimensions are calculated
  • Allow vid progress bar seeker to work when hovering outside player
  • Video player styles
  • Move video controls outside player to fill modal width
  • Clean up a few variable/class names so they are meaningful
  • Resolve bug where newsfeed activity messages are doubled when you click on a modal
  • Media should continue to shrink along vertical axis after you hit the horiz axis min
  • Tablet - touch backdrop to close modal
  • Tablet - Fix overflow on vertical images in portrait orientation
  • Tablet - Fix faux url actually redirecting to media page
  • Tablet - handle video controls
  • Get img element height/width if not already in custom_data
  • Enable horizontal scrolling when window is smaller than modal
  • Replace screen dimensions with hardcoded min/max values
  • Fix failed tests
  • Fix reminded video failure
  • Make videos bigger than intrinsic size
  • Fix reminded images missing permalinks/comments
  • Add message/title content
  • Add feature flag
  • Don't allow click on unloaded video activity

Will close #1539 and #1600

Edited 14 hours ago by Olivia Madrid
Request to merge feat/media-modal-1539 into master
  • Email patches
  • Plain diff
Pipeline #77161100 passed with warnings for bb7450e4 on feat/media-modal-1539
            Merge request approved. Approved by
            Mark Harding
            Mark Harding
            Brian Hatchet
            Brian Hatchet
            Ben Hayward
            Ben Hayward
            Emiliano Balbuena
            Emiliano Balbuena
            Martin Santangelo
            Martin Santangelo
            Marcelo Rivera
            Marcelo Rivera

            Merged by Mark Harding 7 minutes ago

            The changes were merged into master with 13c6f471

            The source branch has been deleted

            Mentions #1539 and #1600

            Pipeline #77255390 canceled for 13c6f471 on master
                          • Discussion 2
                          • Commits 29
                          • Pipelines 18
                          • Changes 36
                          0/1 thread resolved
                          • Loading...
                          • Olivia Madrid @omadrid added 21 commits 3 weeks ago

                            added 21 commits

                            • 76f1573e...3130dd25 - 20 commits from branch master
                            • 811fc6c4 - Merge branch 'master' of gitlab.com:minds/front into feat/media-modal-1539

                            Compare with previous version

                          • Olivia Madrid @omadrid added 1 commit 3 weeks ago

                            added 1 commit

                            • 9dd42d20 - (feat): Media modal - removed a condition

                            Compare with previous version

                          • Olivia Madrid @omadrid changed the description 3 weeks ago

                            changed the description

                          • Olivia Madrid @omadrid mentioned in issue #1539 3 weeks ago

                            mentioned in issue #1539

                          • Olivia Madrid @omadrid added 6 commits 3 weeks ago

                            added 6 commits

                            • 9dd42d20...1790d347 - 4 commits from branch master
                            • b8976fe4 - Merge branch 'master' of gitlab.com:minds/front into feat/media-modal-1539
                            • 7d9266cb - (feat): Media modal

                            Compare with previous version

                          • Olivia Madrid @omadrid added 2 commits 3 weeks ago

                            added 2 commits

                            • e3143161 - 1 commit from branch master
                            • 0a76cb90 - Merge branch 'master' of gitlab.com:minds/front into feat/media-modal-1539

                            Compare with previous version

                          • Mark Harding @markeharding added Squad::Blue scoped label 3 weeks ago

                            added Squad::Blue scoped label

                            • Mark Harding
                              Mark Harding @markeharding · 3 weeks ago
                              Owner

                              I have some funny border colours going on?

                            • Collapse replies
                            • Olivia Madrid
                              Olivia Madrid @omadrid · 2 weeks ago
                              Developer

                              Those are temporary just so I can better see what's happening while fixing the overflow/scroll issues

                            • Please register or sign in to reply
                          • Olivia Madrid @omadrid added 13 commits 2 weeks ago

                            added 13 commits

                            • 0a76cb90...1da91eb3 - 10 commits from branch master
                            • a99e53b3 - (feat): Media modal - minor
                            • 74e16845 - (feat): Media modal - add video component to template
                            • 54b52eb8 - Merge branch 'master' of gitlab.com:minds/front into feat/media-modal-1539

                            Compare with previous version

                            Toggle commit list
                          • Olivia Madrid @omadrid added 1 commit 2 weeks ago

                            added 1 commit

                            • 64a4ffd8 - (feat): Media modal - changed activity to entity

                            Compare with previous version

                          • Olivia Madrid @omadrid added 20 commits 2 weeks ago

                            added 20 commits

                            • 64a4ffd8...4f14088f - 18 commits from branch master
                            • db2714fd - (feat): Media modal - in progress
                            • 1787d182 - Merge branch 'master' of gitlab.com:minds/front into feat/media-modal-1539

                            Compare with previous version

                          • Olivia Madrid @omadrid changed the description 2 times within 6 minutes 2 weeks ago

                            changed the description 2 times within 6 minutes

                          • Olivia Madrid @omadrid changed the description 2 weeks ago

                            changed the description

                          • Olivia Madrid @omadrid added 1 commit 1 week ago

                            added 1 commit

                            • 50ab1ddd - (feat): Media modal - click video auto opens modal

                            Compare with previous version

                          • Olivia Madrid @omadrid added 1 commit 1 week ago

                            added 1 commit

                            • fec7cde9 - (feat): Media modal - prevent reopen modal on video click

                            Compare with previous version

                          • Olivia Madrid @omadrid marked the task Resolve bug where newsfeed activity messages are doubled when you click on a modal as completed 1 week ago

                            marked the task Resolve bug where newsfeed activity messages are doubled when you click on a modal as completed

                          • Emiliano Balbuena @edgebal mentioned in issue #1675 1 week ago

                            mentioned in issue #1675

                          • Olivia Madrid @omadrid marked the task Make hovercard visible in full screen as completed 1 week ago

                            marked the task Make hovercard visible in full screen as completed

                          • Olivia Madrid @omadrid marked the task Media should continue to shrink along opposite axis after you hit the current access min as completed 1 week ago

                            marked the task Media should continue to shrink along opposite axis after you hit the current access min as completed

                          • Olivia Madrid @omadrid marked the task Output min-width of .m-overlay-modal to parent component and apply inline dynamically as completed 1 week ago

                            marked the task Output min-width of .m-overlay-modal to parent component and apply inline dynamically as completed

                          • Olivia Madrid @omadrid marked the task Clean up a few variable/class names so they are meaningful as completed 1 week ago

                            marked the task Clean up a few variable/class names so they are meaningful as completed

                          • Olivia Madrid @omadrid marked the task Enable horizontal scrolling when window is smaller than modal as completed 1 week ago

                            marked the task Enable horizontal scrolling when window is smaller than modal as completed

                          • Olivia Madrid @omadrid marked the task Video player/control styles as completed 1 week ago

                            marked the task Video player/control styles as completed

                          • Olivia Madrid @omadrid marked the task Click video thumbnail to open modal and autoplay as completed 1 week ago

                            marked the task Click video thumbnail to open modal and autoplay as completed

                          • Olivia Madrid @omadrid changed the description 3 times within 3 minutes 1 week ago

                            changed the description 3 times within 3 minutes

                          • Olivia Madrid @omadrid marked the task Enable horizontal scrolling when window is smaller than modal as incomplete 1 week ago

                            marked the task Enable horizontal scrolling when window is smaller than modal as incomplete

                          • Olivia Madrid @omadrid added 7 commits 1 week ago

                            added 7 commits

                            • fec7cde9...79c1eb43 - 6 commits from branch master
                            • eab73fc2 - Merge branch 'master' of gitlab.com:minds/front into feat/media-modal-1539

                            Compare with previous version

                          • Olivia Madrid @omadrid added 1 commit 1 week ago

                            added 1 commit

                            • c1965f0f - (WIP): Media modal - handle video and more

                            Compare with previous version

                          • Olivia Madrid @omadrid changed the description 2 times within 6 minutes 1 week ago

                            changed the description 2 times within 6 minutes

                          • Olivia Madrid @omadrid changed the description 2 times within 2 minutes 1 week ago

                            changed the description 2 times within 2 minutes

                          • Olivia Madrid @omadrid changed the description 3 times within 7 minutes 1 week ago

                            changed the description 3 times within 7 minutes

                          • Olivia Madrid @omadrid marked the task Tablet - Fix faux url actually redirecting to media page as completed 1 week ago

                            marked the task Tablet - Fix faux url actually redirecting to media page as completed

                          • Olivia Madrid @omadrid marked the task Tablet - Fix overflow on vertical images in portrait orientation as completed 1 week ago

                            marked the task Tablet - Fix overflow on vertical images in portrait orientation as completed

                          • Olivia Madrid @omadrid added 8 commits 6 days ago

                            added 8 commits

                            • c1965f0f...14a72a8a - 5 commits from branch master
                            • a3946ec9 - Merge branch 'master' of gitlab.com:minds/front into feat/media-modal-1539
                            • 20f3afe0 - (wip): Media modal - video player controls
                            • ab328cf3 - (wip): Media modal - accommodate theatre

                            Compare with previous version

                            Toggle commit list
                          • Olivia Madrid @omadrid marked the task Tablet - handle video controls as completed 6 days ago

                            marked the task Tablet - handle video controls as completed

                          • Olivia Madrid @omadrid changed the description 5 days ago

                            changed the description

                          • Olivia Madrid @omadrid marked the task Fix failed tests as completed 4 days ago

                            marked the task Fix failed tests as completed

                          • Olivia Madrid @omadrid changed the description 2 times within 1 minute 4 days ago

                            changed the description 2 times within 1 minute

                          • Olivia Madrid @omadrid marked the task Replace screen dimensions with hardcoded min/max values as completed 4 days ago

                            marked the task Replace screen dimensions with hardcoded min/max values as completed

                          • Olivia Madrid @omadrid marked the task Make videos bigger than intrinsic size as completed 4 days ago

                            marked the task Make videos bigger than intrinsic size as completed

                          • Olivia Madrid @omadrid marked the task Enable horizontal scrolling when window is smaller than modal as completed 4 days ago

                            marked the task Enable horizontal scrolling when window is smaller than modal as completed

                          • Olivia Madrid @omadrid added 12 commits 4 days ago

                            added 12 commits

                            • ab328cf3...2e95c013 - 9 commits from branch master
                            • 5596b793 - Merge branch 'master' of gitlab.com:minds/front into feat/media-modal-1539
                            • b5b908ae - (feat): Media modal - minwidth/height calculations
                            • ffeba976 - Merge branch 'master' of gitlab.com:minds/front into feat/media-modal-1539

                            Compare with previous version

                            Toggle commit list
                          • Olivia Madrid @omadrid unmarked as a Work In Progress 4 days ago

                            unmarked as a Work In Progress

                          • Olivia Madrid @omadrid changed the description 4 days ago

                            changed the description

                          • Olivia Madrid @omadrid changed title from (feat): Media modal to [Sprint/KiltedKoala](feat): Media modal 4 days ago

                            changed title from (feat): Media modal to [Sprint/KiltedKoala](feat): Media modal

                          • Olivia Madrid @omadrid marked as a Work In Progress 3 days ago

                            marked as a Work In Progress

                          • Olivia Madrid @omadrid marked the task Get img element height/width if not already in custom_data as incomplete 3 days ago

                            marked the task Get img element height/width if not already in custom_data as incomplete

                          • Olivia Madrid @omadrid marked the task Get img element height/width if not already in custom_data as completed 3 days ago

                            marked the task Get img element height/width if not already in custom_data as completed

                          • Olivia Madrid @omadrid changed the description 2 days ago

                            changed the description

                          • Olivia Madrid @omadrid added 15 commits 2 days ago

                            added 15 commits

                            • ffeba976...268165b1 - 12 commits from branch master
                            • 9148a391 - (feat): Media modal - handle images without custom data height/widhts
                            • 99e25344 - Merge branch 'master' of gitlab.com:minds/front into feat/media-modal-1539
                            • 215074b8 - (feat): Media modal - fix broken permalinks and comments for certain reminds

                            Compare with previous version

                            Toggle commit list
                          • Olivia Madrid @omadrid marked the task Fix reminded images missing permalinks/comments as completed 2 days ago

                            marked the task Fix reminded images missing permalinks/comments as completed

                          • Olivia Madrid @omadrid added 1 commit 1 day ago

                            added 1 commit

                            • 44e3c610 - (feat): Media modal - show activity message in modal

                            Compare with previous version

                          • Olivia Madrid @omadrid unmarked as a Work In Progress 1 day ago

                            unmarked as a Work In Progress

                          • Olivia Madrid @omadrid marked the task Fix reminded images missing permalinks/comments as incomplete 1 day ago

                            marked the task Fix reminded images missing permalinks/comments as incomplete

                          • Olivia Madrid @omadrid marked the task Add message/title content as completed 1 day ago

                            marked the task Add message/title content as completed

                          • Olivia Madrid @omadrid marked the task Fix reminded images missing permalinks/comments as completed 1 day ago

                            marked the task Fix reminded images missing permalinks/comments as completed

                          • Olivia Madrid @omadrid marked as a Work In Progress 16 hours ago

                            marked as a Work In Progress

                          • Olivia Madrid @omadrid changed the description 16 hours ago

                            changed the description

                          • Olivia Madrid @omadrid added 1 commit 15 hours ago

                            added 1 commit

                            • d986f5b2 - (feat): Media modal - add feature flag

                            Compare with previous version

                          • Olivia Madrid @omadrid marked the task Add feature flag as completed 14 hours ago

                            marked the task Add feature flag as completed

                          • Olivia Madrid @omadrid unmarked as a Work In Progress 14 hours ago

                            unmarked as a Work In Progress

                          • Olivia Madrid @omadrid changed the description 14 hours ago

                            changed the description

                          • Olivia Madrid @omadrid marked as a Work In Progress 14 hours ago

                            marked as a Work In Progress

                          • Olivia Madrid @omadrid changed the description 14 hours ago

                            changed the description

                          • Olivia Madrid @omadrid added 1 commit 14 hours ago

                            added 1 commit

                            • bb7450e4 - (feat): Media modal - don't allow click on unloaded videos

                            Compare with previous version

                          • Olivia Madrid @omadrid marked the task Don't allow click on unloaded video activity as completed 14 hours ago

                            marked the task Don't allow click on unloaded video activity as completed

                          • Olivia Madrid @omadrid unmarked as a Work In Progress 14 hours ago

                            unmarked as a Work In Progress

                          • Olivia Madrid @omadrid changed the description 14 hours ago

                            changed the description

                          • Mark Harding @markeharding approved this merge request 9 minutes ago

                            approved this merge request

                          • Mark Harding @markeharding merged 7 minutes ago

                            merged

                          • Mark Harding @markeharding mentioned in commit 13c6f471 7 minutes ago

                            mentioned in commit 13c6f471

                          • You're only seeing other activity in the feed. To add a comment, switch to one of the following options.
                          Please register or sign in to reply
                          0 Assignees
                          None
                          Assign to
                          None
                          Milestone
                          None
                          Assign milestone
                          None
                          Time tracking
                          No estimate or time spent
                          1
                          Labels
                          Squad::Blue
                          Assign labels
                          • View project labels
                          Lock merge request
                          Unlocked
                          7
                          7 participants
                          user avatar
                          Brian Hatchet
                          user avatar
                          Ben Hayward
                          user avatar
                          Emiliano Balbuena
                          user avatar
                          Martin Santangelo
                          user avatar
                          Marcelo Rivera
                          user avatar
                          Mark Harding
                          user avatar
                          Olivia Madrid
                          Reference: minds/front!467

                          Revert this merge request

                          This will create a new commit in order to revert the existing changes.

                          Switch branch
                          Cancel
                          A new branch will be created in your fork and a new merge request will be started.

                          Cherry-pick this merge request

                          Switch branch
                          Cancel
                          A new branch will be created in your fork and a new merge request will be started.