feat: Exposes the max enabled resolution in the connection stats popover. (#7278)

* feat: Exposes the max enabled resolution in the connection stats popover.
* deps: lib-jitsi-meet@latest
This commit is contained in:
George Politis 2020-07-13 18:20:59 +03:00 committed by GitHub
parent b0188a7184
commit e4ce3928dc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 18 additions and 3 deletions

View File

@ -110,6 +110,7 @@
"localaddress_plural": "Local addresses:", "localaddress_plural": "Local addresses:",
"localport": "Local port:", "localport": "Local port:",
"localport_plural": "Local ports:", "localport_plural": "Local ports:",
"maxEnabledResolution": "send max",
"more": "Show more", "more": "Show more",
"packetloss": "Packet loss:", "packetloss": "Packet loss:",
"quality": { "quality": {

View File

@ -56,7 +56,7 @@
"jquery-i18next": "1.2.1", "jquery-i18next": "1.2.1",
"js-md5": "0.6.1", "js-md5": "0.6.1",
"jwt-decode": "2.2.0", "jwt-decode": "2.2.0",
"lib-jitsi-meet": "github:jitsi/lib-jitsi-meet#cd008d726f1f57562eb5d8e6a3cd91c7e69826a0", "lib-jitsi-meet": "github:jitsi/lib-jitsi-meet#304b0a2b4e18216d792f499c74fc24bc3849303e",
"libflacjs": "github:mmig/libflac.js#93d37e7f811f01cf7d8b6a603e38bd3c3810907d", "libflacjs": "github:mmig/libflac.js#93d37e7f811f01cf7d8b6a603e38bd3c3810907d",
"lodash": "4.17.13", "lodash": "4.17.13",
"moment": "2.19.4", "moment": "2.19.4",

View File

@ -342,6 +342,7 @@ class ConnectionIndicator extends AbstractConnectionIndicator<Props, State> {
bridgeCount, bridgeCount,
e2eRtt, e2eRtt,
framerate, framerate,
maxEnabledResolution,
packetLoss, packetLoss,
region, region,
resolution, resolution,
@ -358,6 +359,7 @@ class ConnectionIndicator extends AbstractConnectionIndicator<Props, State> {
e2eRtt = { e2eRtt } e2eRtt = { e2eRtt }
framerate = { framerate } framerate = { framerate }
isLocalVideo = { this.props.isLocalVideo } isLocalVideo = { this.props.isLocalVideo }
maxEnabledResolution = { maxEnabledResolution }
onShowMore = { this._onToggleShowMore } onShowMore = { this._onToggleShowMore }
packetLoss = { packetLoss } packetLoss = { packetLoss }
region = { region } region = { region }

View File

@ -57,6 +57,12 @@ type Props = {
*/ */
isLocalVideo: boolean, isLocalVideo: boolean,
/**
* The send-side max enabled resolution (aka the highest layer that is not
* suspended on the send-side).
*/
maxEnabledResolution: number,
/** /**
* Callback to invoke when the show additional stats link is clicked. * Callback to invoke when the show additional stats link is clicked.
*/ */
@ -380,8 +386,8 @@ class ConnectionStatsTable extends Component<Props> {
* @returns {ReactElement} * @returns {ReactElement}
*/ */
_renderResolution() { _renderResolution() {
const { resolution, t } = this.props; const { resolution, maxEnabledResolution, t } = this.props;
const resolutionString = Object.keys(resolution || {}) let resolutionString = Object.keys(resolution || {})
.map(ssrc => { .map(ssrc => {
const { width, height } = resolution[ssrc]; const { width, height } = resolution[ssrc];
@ -389,6 +395,12 @@ class ConnectionStatsTable extends Component<Props> {
}) })
.join(', ') || 'N/A'; .join(', ') || 'N/A';
if (maxEnabledResolution && maxEnabledResolution < 720) {
const maxEnabledResolutionTitle = t('connectionindicator.maxEnabledResolution');
resolutionString += ` (${maxEnabledResolutionTitle} ${maxEnabledResolution}p)`;
}
return ( return (
<tr> <tr>
<td> <td>