629 Commits (5e01abf6fbefe882448e5e133f3a7055438546ac)

Author	SHA1	Message	Date
Lyubo Marinov	8cdd73b987	[RN] Reverts to own properties for Storage	8 years ago
Lyubo Marinov	a7ee632f43	[RN] LocalVideoTrackUnderlay ... Implement a React Component which displays children as an overlay of local video. The WelcomePage implemented such a component inside of it among other WelcomePage-specific logic so I split LocalVideoTrackUnderlay out of it. The new Component is used on the BlankPage which may be displayed in the future not only when the WelcomePage is disabled but also when there are long running network requests, for example.	8 years ago
Lyubo Marinov	b304ad5808	[RN] Weaken the coupling between WelcomePage and AbstractWelcomePage	8 years ago
Lyubo Marinov	86e4876df2	Coding style	8 years ago
Saúl Ibarra Corretgé	034518a6a0	[RN] Cache configurations in localStorage ... This only helps iff there is a short transient network error which prevents the configuration from being loaded. In such case, use the cached version in localStorage, which may not match the shard, but it's (probably!) better than nothing. In case there is no Internet connectivity, an error will be produced as soon as the XMPP connection is attempted anyway.	8 years ago
Lyubo Marinov	bf523711df	[RN] Prepare to polyfill sessionStorage	8 years ago
Saúl Ibarra Corretgé	bfeaf329e1	[RN] Add polyfill for localStorage ... It's built on top of React Native's AsyncStorage. They have differing APIs, so we implement a synchronous API on top of an asynchronous one. This is done by being optimistic and hoping that operations will happen asynchronously. If one such operation fails, the error is ignored and life goes on, since operations are performed in the in-memory cache first. Note to reviewers: LocalStorage.js lacks Flow annotations because indexable class declarations are not yet supported: https://github.com/facebook/flow/issues/1323 and yours truly couldn't find a way to make the required syntax work without making it unnecessarily complex.	8 years ago
Saúl Ibarra Corretgé	8fc095039e	analytics: lower log severity when we fail to create a handler ... It'snot the end of the world, we can move on.	8 years ago
Saúl Ibarra Corretgé	eca04de348	Simplified code	8 years ago
Saúl Ibarra Corretgé	18d1572dab	[RN] Load config.js only when the room is known ... This patch loads the config later than we used to, that is, only once we know the room the user is about to join. Due to architectural limitations in lib-jitsi-meet, it needs to be initialized with a configuration in order to properly function. This is unfortunate because we need to create a video track in the welcome page, but don't know the room (hence no config) yet. In order to circumvent this problem an empty configuration is used, which is later swapped with the appropriate one, once loaded. Some interesting side-effects of this change are a perceived speed increase when the app starts or a conference is hangup. They are both due to the fact that no config needs to be fetched from a remote server in those cases.	8 years ago
Lyubo Marinov	45a1ae26ca	[RN] Prepare to display BlankPage more ... For example, while config.js and other files are being loaded before the navigation to Conference is feasible.	8 years ago
Lyubo Marinov	6545a7a1bb	Remove duplication	8 years ago
Lyubo Marinov	ec9c05e401	[RN] Load config.js with ?room= ... In order to load the configuration from the shard that will actually host the conference, it's imperative that we add the room= query parameter: https://meet.jit.si/config.js?room=example This implies a departure from our current model, where the config is discarded if the domain for the next conference is different, but kept otherwise.	8 years ago
virtuacoplenny	bf03e73876	feat(filmstrip): show thumbnails with toolbar and on hover (#1944) ... * feat(filmstrip): show thumbnails with toolbar and on hover * squash: reduce verbosity of logic for when to display * squash: remove check for fake participant Before fake participant (youtube video) would make the filmstrip always displayed. However, youtube videos already dock the toolbar, so filmstrip will remain displayed, so the check is redundant. * squash: change mouse hover listener targets	8 years ago
Дамян Минков	6682543691	Moves analytics loading to react. (#1945) ... * feat(analytics): move to React The analytics handlers have been moved to JitsiMeetGlobalNS, so now they are stored in `window.JitsiMeetJS.app.analyticsHandlers`. The analytics handlers are re-downloaded and re-initialized on every lib-jitsi-meet initialization, which happens every time the config is changed (moving between deployments in the mobile app). * Adds legacy support for old analytics location.	8 years ago
Lyubo Marinov	8436f23e05	Comments	8 years ago
hristoterezov	cd3dad956b	fix(peopleSearch): queryTypes are defined in config.js	8 years ago
Saúl Ibarra Corretgé	328da08b3a	[RN] Remove obsolete comment ... We have already made the implicit decision not to pursue what the comment describes. If we ever revisit it, it probably won't be handled where the comment is anyway.	8 years ago
Leonard Kim	05f8c69fe6	ref(toolbar): add wrapper around buttons for torture tests ... The torture tests were looking for the anchor tag within each button. However, that anchor could get blown away from a react re-render. So instead, expose a way for the torture tests to find the root node of the button.	8 years ago
Leonard Kim	7063f144ef	feat(tooltips): add for video quality label and feedback button	8 years ago
Leonard Kim	5e4f921e1b	ref(video-quality-label): do not show quality dialog	8 years ago
Lyubo Marinov	7f8e8177d0	[RN] Refactor "Keep track of ongoing network requests" and "Show a progress indicator in the BlankPage" ... I'm not saying that the two commits in question were wrong or worse than what I'm offering. Anyway, I think what I'm offering brings: * Compliance with expectations i.e. the middleware doesn't compute the next state from the current state, the reducer does; * Clarity and/or simplicity i.e. there's no global variable (reqIndex), there's no need for the term "index" (a.k.a "reqIndex") in the redux store. * By renaming net-interceptor to network-activity feels like it's preparing the feature to implement a NetworkActivityIndicator React Component which will take on more of the knowledge about the specifics of what is the network activity redux state exactly, is it maintained by interception or some other mechanism, and abstracts it in the feature itself allowing outsiders to merely render a React Component.	8 years ago
Saúl Ibarra Corretgé	e33030582f	[RN] Show a progress indicator in the BlankPage ... It will only be shown when there are ongoing network requests.	8 years ago
Saúl Ibarra Corretgé	d669a6c73c	[RN] Keep track of ongoing network requests ... Works only for XHR requests, which is the only network request mobile performs (WebRTC traffic aside). The fetch API is implemented on top of XHR, so that's covered too. Requests are kept in the redux store until they complete, at which point they are removed.	8 years ago
Saúl Ibarra Corretgé	8eebfcad72	[RN] Add support for callstats on mobile ... The global.performance.now polyfill is removed since it's polyfilled by react-native-callstats.	8 years ago
virtuacoplenny	ef1b8fdb77	ref(1-on-1): move remote video visibility to a selector (#1922) ... * ref(1-on-1): move remote visibility to a selector Derive whether or not remote videos should display using a selector to look across different states. A selector was chosen over using memoized selectors (reselect) or subscribers as a first step approach, avoiding additional mutations caused by a subscriber updating the filmstrip state and avoiding additional api overhead introduced by reselect. * rename selector	8 years ago
Saúl Ibarra Corretgé	c0f648b1ab	[RN] Remove no longer needed fetch API fallback	8 years ago
Saúl Ibarra Corretgé	531b638a8a	[RN] Switch default avatars to meeples	8 years ago
bbaldino	f616b0b71b	few tweaks to fix some exceptions in edge	8 years ago
hristoterezov	70422f4a47	fix(alwaysontop): change the size of the toolbar	8 years ago
virtuacoplenny	735a596afe	ref(recording): convert recording label to react (#1915) ... * ref(recording): convert recording label to react - Create a RecordingLabel component for displaying the current recording state, as reflected in the redux store. This is needed for 1-on-1 mode to be completely in redux. - Update the store with the recording state so RecordingLabel can update itself. - Remove previous logic for updating the non-react label, which includes event emitting for filmstrip visibility changes, as RecordingLabel is hooked into redux updates. * ref(recording): use status and type constants from lib * make label really dumb, move logic back to Recording	8 years ago
Leonard Kim	85f0ad2791	feat(popover): create a wrapper around InlineDialog	8 years ago
Leonard Kim	fdee6dc360	fix(inline-dialog): reimplement popover display on mouse move ... Create empty elements within InlineDialog content that can be used to bridge mouse movement from the InlineDialog trigger to the InlineDialog content. The empty elements are positioned absolute so they can break out of the InlineDialog container and not affect popper's position calculations.	8 years ago
Lyubo Marinov	24db52ef0f	[RN] Remove denied permission alert from WelcomePage	8 years ago
Lyubo Marinov	0b8c12de0e	Simplify route navigation ... I see it as the first step in simplifying the route navigate of the JavaScript app by removing BlankWelcomePage from _getRouteToRender. From a faraway point of view, the app is at the route at which it is not in a conference. Historically, the route was known as the Welcome page. But mobile complicated the route by saying that actually it may not want to see the room name input and join button. Additionally, I renamed BlankWelcomePage to BlankPage because I don't think of it as a WelcomePage alternative but rather as a more generic BlankPage which may be utilized elsewhere in the future. I plan for the next steps to: * Merge Entryway, _interceptComponent, and _getRouteToRender in one React Component rendered by AbstractApp so that the whole logic is in one file; * Get rid of RouteRegistry and routes.	8 years ago
Lyubo Marinov	cb5b93fb6e	Switch to prop-types	8 years ago
Lyubo Marinov	7114614697	Remove obsolete workaround	8 years ago
Leonard Kim	ec22329408	ref(local-video): generate html instead of hardcoding in filmstrip ... This would better mimic React behavior where each component has its own markup, instead of relying on markup existing elsewhere.	8 years ago
Saúl Ibarra Corretgé	26f0f7f89c	[RN] Alert the user when they need to manually grant a permission	8 years ago
Saúl Ibarra Corretgé	9bca0e3b3d	[RN] Create tracks right when they are required ... When do we need tracks? - Welcome page (only the video track) - Conference (depends if starting with audio / video muted is requested) When do we need to destroy the tracks? - When we are not in a conference and there is no welcome page In order to accommodate all the above use cases, a new component is introduced: BlankWelcomePage. Its purpose is to take the place of the welcome page when it is disabled. When this component is mounted local tracks are destroyed. Analogously, a video track is created when the (real) welcome page is created, and all the desired tracks are created then the Conference component is created. What are desired tracks? These are the tracks we'd like to use for the conference that is about to happen. By default both audio and video are desired. It's possible, however, the user requested to start the call with no video/audio, in which case it's muted in base/media and a track is not created. The first time the app starts (with the welcome page) it will request permission for video only, since there is no need for audio in the welcome page. Later, when a conference is joined permission for audio will be requested when an audio track is to be created. The audio track is not destroyed when the conference ends. Yours truly thinks this is not needed since it's a stopped track which is not using system resources.	8 years ago
Saúl Ibarra Corretgé	3102ea6818	[RN] Try to create local tracks when unmuting, if track is missing ... This is only desired when the unmuting action took place due to a manual user intervention or the audio-only mode being disengaged.	8 years ago
Leonard Kim	28b4595561	ref(tooltips): remove old tooltips	8 years ago
Leonard Kim	e3361e2f3b	feat(tooltips): convert popup tooltips to InlineDialog	8 years ago
Leonard Kim	c3a4a38414	feat(toolbar): use AtlasKit tooltip	8 years ago
Leonard Kim	38e2443ab7	feat(small-video): use AtlasKit tooltip	8 years ago
Saúl Ibarra Corretgé	c42f1704ff	[RN] Rename createInitialLocalTracks to createLocalTracks ... The name better suits its purpose, since it can be called at any time.	8 years ago
Leonard Kim	5358f022ff	fix(config): override config values instead of merging ... Iterate over objects and copy over primitives and arrays instead of using _.merge, as merge will not replace a config entry completely. For arrays in a target object, the arrays will have its indices replaced. This means if a source array is empty, the target array will be left alone. Similarly, if the target array is longer than a source array, there will be indices not touched in the target array.	8 years ago
hristoterezov	5ef914602f	fix(remotecontrol): Handle screen sharing cancel	8 years ago
Lyubo Marinov	2818520c8f	[RN] Make react-native-img-cache optional at bundle time	8 years ago
Lyubo Marinov	131e5af01e	[RN] Fix a strange import of a feature by itself	8 years ago