Modal pager
0/1
thread resolved
Summary
Go back and forth on images/videos/blogs for the current user or group on media modal.
Closes #1737.
Steps to test
- Visit https://epic-modal-pager.minds.io/0xemi/. Arrows will appear in the main view to allow going back and forth all user's media.
- Log in with your user
- Post 2 or more images or blogs (video transcoder seems to be down for review sites)
- Click one of them, you should be able to go back and forth using the arrows (keyboard/on-screen element)
- Test that you are able to close the media modal using Esc key.
- Test that when you use the keyboard arrows on text fields (i.e. the comment input) it doesn't trigger the navigation
- Go to a group (or create a new one), copy the URL
- Post a couple of images
- Login with another user (or create a new one)
- Go to the group and join it
- Post a couple of images
- Test that you can go back and forth all groups images on the media modal.
Estimated Regression Scope
- Media modal entity loading
- Media modal media sizing
- Media modal permalink URLs
Request to merge
epic/modal-pager
into
master
added scoped label
mentioned in merge request engine!437
changed the description
added 1 commit
- e6fe8635 - (feat): Keyboard navigation + esc key handling
added 1 commit
- a8729b07 - (feat): Change dimensions on-the-fly when video metadata is loaded
added 10 commits
-
3a92f989...1c8ee565 - 8 commits from branch
master - d6e85f23 - Merge remote-tracking branch 'origin/master' into epic/modal-pager
- a0cf2bb6 - (refactor): ditch redirectUrl and handle URL hot-swap on modal
-
3a92f989...1c8ee565 - 8 commits from branch
changed the description
added scoped label
assigned to @benhayward.ben
unmarked as a Work In Progress
![Ben Hayward](data:image/svg+xml,<svg xmlns="http://www.w3.org/2000/svg" width="200" height="200"><rect fill-opacity="0"/></svg>)
Last updated by Ben Hayward1 import { Injectable } from '@angular/core'; 2 import { Client } from '../../services/api/client'; 3 import { EntitiesService } from './entities.service'; 4 5 export type HorizontalFeedContext = 'container'; 6 7 @Injectable() - Developer
It's self explanatory enough, but some class and function level docs here may save someone some time in the future.
- Developer
Nice docs, thanks Emi!
added 6 commits
-
a0cf2bb6...1181bc82 - 4 commits from branch
master - e26eefec - Merge remote-tracking branch 'origin/master' into epic/modal-pager
- bd72b1f8 - (chore): JSdoc
-
a0cf2bb6...1181bc82 - 4 commits from branch
resolved all threads
