jitsi-meet

Commit Graph

Author	SHA1	Message	Date
virtuacoplenny	1984f8d0c0	fix(vertical-filmstrip): scope previous Firefox scroll fixes (#2276 ) In `aa93a783`, remote videos not scrolling on Firefox, IE, and Edge was worked around using CSS. The CSS selector is too vague and also catches horizontal filmstrip. The selector should apply to vertical filmstrip only.	8 years ago
Leonard Kim	a653816f90	fix(vertical-filmstrip): prevent shared videos from displaying under avatars - Modify Etherpad and SharedVideo so their resizing takes into account the width of the filmstrip in vertical filmstrip mode. - Modify Filmstrip's getFilmstripWidth to account for when the filmstrip is hidden. - modify VideoContainer so in vertical filmstrip mode it centers the shared desktop stream in the middle of the available space not taken by filmstrip. - Also allow clickthrough on the secondary toolbar itself while still allowing clicks on the toolbar's buttons. This allows clicks on shared videos to go through.	8 years ago
Leonard Kim	40df5f97d4	fix(vertical-filmstrip): no scrollbars on Firefox from single thumbnail Something is causing firefox to show scrollbars on the remote videos container even though there is plenty of room. I lost this battle and just made the container taller...	8 years ago
Leonard Kim	aa93a78372	fix(vertical-filmstrip): allow overflow scrolling on firefox, edge, and ie Using column-reverse prevents proper scrolling on browsers other than Safari and Chrome. Additionally, Firefox has an issue where flex containers have dimensions set to auto, preventing resize. So, add hacks to maintain Chrome and Safari's behavior while allowing for some kind of scrolling on other browsers.	8 years ago
Leonard Kim	c345c0d5e8	fix(filmstrip): do not apply flex on local video for horizontal mode Firefox flex behaves differently from chrome. In firefox, flex is not allowing the local video and invite button to exceed the height of the parent, causing the local video to display shorter than expected due to the invite button. Flex doesn't need to be applied for local video in horizontal filmstrip mode as it's only needed in vertical filmstrip mode to flip the position of the invite button from above the local video to below. Honestly, this could be better off done declaratively in the html, but that's not as straigtforward in this jquery+react world.	8 years ago
Leonard Kim	a8cc65ab9a	fix(filmstrip-only): do not re-align local video for invite button The bottom alignment of the local video container is adjusted in horizontal filmstrip mode so that the videos remain evenly aligned at the top but the invite button sticks out. With the invite button hidden in filmstrip only, this only caused misalignment. Re-adjust the local video container back down and put transitions on the filmstrip wrapper itself, instead of on the thumbnail wrapper level to avoid transitions when re-adjusting.	8 years ago
yanas	86fcfcc535	WiP(invite-ui): Initial move of invite UI to invite button (#1950 ) * WiP(invite-ui): Initial move of invite UI to invite button * Adjusts styling to fit both horizontal and vertical filmstrip * Removes comment and functions not needed * [squash] Addressing various review comments * [squash] Move invite options to a separate config * [squash] Adjust invite button styles until we fix the whole UI theme * [squash] Fix the remote videos scroll * [squash]:Do not show popup menu when 1 option is available * [squash]: Disable the invite button in filmstrip mode * feat(connection-indicator): implement automatic hiding on good connection (#2009) * ref(connection-stats): use PropTypes package * feat(connection-stats): display a summary of the connection quality * feat(connection-indicator): show empty bars for interrupted connection * feat(connection-indicator): change background color based on status * feat(connection-indicator): implement automatic hiding on good connection * fix(connection-indicator): explicitly set font size Currently non-react code will set an icon size on ConnectionIndicator. This doesn't work on initial call join in vertical filmstrip after some changes to support hiding the indicator. The chosen fix is passing in the icon size to mirror what would happe with full filmstrip reactification. * ref(connection-stats): rename statuses * feat(connection-indicator): make hiding behavior configurable The original implementation made the auto hiding of the indicator configured in interfaceConfig. * fix(connection-indicator): readd class expected by torture tests * fix(connection-indicator): change connection quality display styling Bold the connection summary in the stats popover so it stands out. Change the summaries so there are only three--strong, nonoptimal, poor. * fix(connection-indicator): gray background on lost connection * feat(icons): add new gsm bars icon * feat(connection-indicator): use new 3-bar icon * ref(icons): remove icon-connection and icon-connection-lost Both have been replaced by icon-gsm-bars so they are not being referenced anymore. Mobile looks to have connect-lost as a separate icon in font-icons/jitsi.json. * fix(defaultToolbarButtons): Fixes unresolved InfoDialogButton component problem * [squash]: Makes invite button fit the container * [squash]:Addressing invite truncate, remote menu position and comment * [squash]:Fix z-index in horizontal mode, z-index in lonely call * [squash]: Fix filmstripOnly property, remove important from css	8 years ago
Leonard Kim	c54879d605	fix(video-quality-label): push around z-indexes for firefox	8 years ago
Leonard Kim	38e2443ab7	feat(small-video): use AtlasKit tooltip	8 years ago
Leonard Kim	5e4d3de8fd	fix(filmstrip): re-adjust z-indexes for tooltip display	8 years ago
Leonard Kim	995a25ee15	fix(video-quality-label): adjust z-indexing clashes with filmstrip With AtlasKit InlineDialog, the filmstrip z-index in vertical filmstrip mode was adjusted as it had to be set to fixed position for the dialogs to automagically be position fixed. This left behind a regression where the filmstrip z-index was the same as the video quality label, but came later in the dom, so the label was not clickable. This commit modifies the z-index of the vertical filmstrip to account for clicking of the video quality label.	8 years ago
virtuacoplenny	725d39ddcd	feat(jitsipopover): convert to InlineDialog (#1804 ) * feat(small-video): use InlineDialog for stats and remote menu - Remove JitsiPopover and use InlineDialog instead. - Bring the remote menu icon into react. - Make vertical filmstrip position:fixed so popper (AtlasKit dependency) sets InlineDialogs and eventually tooltips to position:fixed. * ref(remote-menu): hook KickButton to redux * ref(remote-menu): hook MuteButton to redux * modify padding, toggle dialogs * pixel push margins to align dialogs, adjust padding of dialogs * add comment about margin for dialog, add file I forgot * modify indicator markup so the icon can be moved down while trigger stays at top of toolbar	8 years ago
virtuacoplenny	da1c760abf	feat(notifications): implement a react/redux notification system (#1786 ) * feat(notifications): implement a react/redux notification system * squash into impl explicit timeout, style * ref(notifications): convert toastr notifications to use react * ref(toastr): remove library * squash into conversion: pass timeout * squash into clean remove from debian patch	8 years ago
yanas	2e2b1d47c0	Changes video thumbnail menu icon	8 years ago
virtuacoplenny	84ae7df8f1	fix(filmstrip-only): vertically align center the toolbar (#1700 ) * fix(filmstrip-only): vertically align center the toolbar Use top 50% to position the toolbar's top at the vertical center of the iframe. Then use transform 50% to move the toolbar itself up 50% so its middle matches the middle of the iframe. * squash: toolbox should center with filmstrip	8 years ago
Leonard Kim	ae67b2b28e	fix(vertical-filmstrip): enable hardware acceleration for icons Icons on the thumbnails can flicker when scrolling through videos. To give rendering a bit more power, and thereby help with rendering icons without flickering, force hardware acceleration.	8 years ago
Leonard Kim	152427e01b	feat(small-video): convert the "toolbar" to react Move display of audio muted, video muted, and moderator icons, which make up the elements of the small video toolbar, into React Components.	8 years ago
Leonard Kim	1a87ee5f93	fix(filmstrip): adjust filmstrip remote videos positioning - For horizontal mode, remove extra spacing created by borders around local video and remote videos. - For vertical mode, ensure remote videos grow only to fill the parent height.	8 years ago
virtuacoplenny	d5b40280ab	fix(vertical-filmstrip): different label animations for filmstrip states (#1596 ) * fix(vertical-filmstrip): different label animations for filmstrip states Instead of one timing for sliding the video status label left and right, have different timings depending on the filmstrip state. To facilitate triggering the different animations, add more classes to the labels that need to move that specify the filmstrip state. - Faster transition if focusing on self-view with videos present so the label does not overlap videos transitioning from 0 opacity. - Transition delay when de-focusing on self-view with videos present so videos have time to go away before the label moves over them. - Maintain no movement if there are no videos, regardless of filmstrip toggle state. - Different delays for when the filmstrip is being toggled visible and hidden if there are remote videos visible. * SQUASH: remove remote videos count * SQUASH: add docs to scss	8 years ago
Leonard Kim	acb4d12928	fix(vertical-filmstrip): do not move recording label if not in the corner	8 years ago
Leonard Kim	56b12bd969	fix(vertical-filmstrip): move video status labels back to top right The video status labels, which include recording and hd status, have been moved back to the top left while in vertical filmstrip mode. The following had to be done: - Remove styling to move the labels to the bottom left - For VideoStatusLabel, move filmstrip remote video count, toggle state, and 1:1 state into redux. - Use middleware to emit out to the Recording label when the filmstrip changes. - Create an empty Filmstrip file for web and identify the existing Filmstrip component as native.	8 years ago
Leonard Kim	aabe641047	feat(vertical-filmstrip): Initial implementation - Add a class to the body when in vertical filmstrip mode - Override styles as necessary to support the mode - Add an option to make tooltips display from the left - Move the HD Label to the bottom left - Move the remote video menu to the bottom left, move the mute icons to the bottom right - Scale the local video's height and width to fit the filmstrip	8 years ago

22 Commits (c4468cb7b80349428c0ab545085e1760c210cbd3)