Creepso/stremio-web
1
0
Fork 0
mirror of https://github.com/Stremio/stremio-web.git synced 2026-08-19 22:57:51 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

NavBar uses React.memo for oprimization

Browse source
This commit is contained in:
NikolaBorislavovHristov 2019-04-25 23:01:07 +03:00
parent a586252ad0
commit 23239b0847
1 changed files with 4 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/common/NavBar/NavBar.js
View file

@ -5,7 +5,7 @@ const SearchBar = require('./SearchBar');
const UserMenu = require('./UserMenu');
const styles = require('./styles');
const NavBar = ({ backButton, tabs, title, searchBar, userMenu }) => {
const NavBar = React.memo(({ backButton, tabs, title, searchBar, userMenu }) => {
const onBackButtonClick = React.useCallback(() => {
window.history.back();
}, []);
@ -23,7 +23,7 @@ const NavBar = ({ backButton, tabs, title, searchBar, userMenu }) => {
null
}
{
tabs.length > 0 ?
Array.isArray(tabs) && tabs.length > 0 ?
<div className={styles['nav-tabs-container']}>
{tabs.map(({ href, icon, label }) => (
<NavBarButton
@ -52,8 +52,9 @@ const NavBar = ({ backButton, tabs, title, searchBar, userMenu }) => {
}
</nav>
);
};
});
NavBar.displayName = 'NavBar';
NavBar.propTypes = {
tabs: PropTypes.arrayOf(PropTypes.exact({
icon: PropTypes.string.isRequired,
@ -66,12 +67,4 @@ NavBar.propTypes = {
userMenu: PropTypes.bool.isRequired
};
NavBar.defaultProps = {
tabs: Object.freeze([]),
title: '',
backButton: false,
searchBar: false,
userMenu: false
};
module.exports = NavBar;