LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

fix 1972

Browse source 
Adds a snackbar if the lbry:// URL is invalid
I tried to do the doNotify similar to most of the other pages, but failed.
This commit is contained in:
Thomas Zarebczan 2018-10-11 01:56:47 -04:00
parent 352363056c
commit 2707210744
1 changed files with 25 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
src/renderer/component/wunderbar/view.jsx
View file

@ -1,7 +1,7 @@
// @flow
import React from 'react';
import classnames from 'classnames';
import { normalizeURI, SEARCH_TYPES } from 'lbry-redux';
import { normalizeURI, SEARCH_TYPES, doNotify, isURIValid } from 'lbry-redux';
import Icon from 'component/common/icon';
import { parseQueryParams } from 'util/query_params';
import * as icons from 'constants/icons';
@ -101,9 +101,18 @@ class WunderBar extends React.PureComponent<Props> {
if (suggestion.type === 'search') {
onSearch(query, resultCount);
} else {
if (isURIValid(query)) {
const params = getParams();
const uri = normalizeURI(query);
onSubmit(uri, params);
} else {
window.app.store.dispatch(
doNotify({
message: __('Invalid LBRY URL requested. Only A-Z, a-z, and - allowed.'),
displayType: ['snackbar'],
})
);
}
}
return;
@ -112,9 +121,18 @@ class WunderBar extends React.PureComponent<Props> {
// Currently no suggestion is highlighted. The user may have started
// typing, then lost focus and came back later on the same page
try {
if (isURIValid(query)) {
const uri = normalizeURI(query);
const params = getParams();
onSubmit(uri, params);
} else {
window.app.store.dispatch(
doNotify({
message: __('Invalid LBRY URL entered. Only A-Z, a-z, and - allowed.'),
displayType: ['snackbar'],
})
);
}
} catch (e) {
onSearch(query, resultCount);
}