iop
/
superset
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[fix] Support APP_ICON_WIDTH configuration parameter in SPA menu (#9666) 

* Expose additional APP_ICON_WIDTH python config variable to ui via app bootstrap for use in Menu

* Update base.py to inclue APP_ICON_WIDTH via bracket notation instead of .get and adjust Menu proptypes to be required.

Co-authored-by: Natalie Ruhe <nruhe@nruhe.attlocal.net>
Co-authored-by: Natalie Ruhe <natalie@preset.io>
This commit is contained in:
Natalie Ruhe 2020-04-28 12:59:03 -07:00 committed by GitHub
parent a52cfcd234
commit 9d755bc1ad
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
superset-frontend/spec/javascripts/components/Menu_spec.jsx
View File

@ -75,6 +75,7 @@ const defaultProps = {
path: '/superset/profile/admin/',
icon: '/static/assets/images/superset-logo@2x.png',
alt: 'Superset',
width: '126',
},
navbar_right: {
bug_report_url: null,

3
superset-frontend/src/components/Menu/Menu.jsx
View File

@ -33,6 +33,7 @@ const propTypes = {
path: PropTypes.string.isRequired,
icon: PropTypes.string.isRequired,
alt: PropTypes.string.isRequired,
width: PropTypes.string.isRequired,
}).isRequired,
navbar_right: PropTypes.shape({
bug_report_url: PropTypes.string,
@ -59,7 +60,7 @@ export default function Menu({
<Navbar.Header>
<Navbar.Brand>
<a className="navbar-brand" href={brand.path}>
<img width="126" src={brand.icon} alt={brand.alt} />
<img width={brand.width} src={brand.icon} alt={brand.alt} />
</a>
</Navbar.Brand>
<Navbar.Toggle />

1
superset/views/base.py
View File

@ -241,6 +241,7 @@ def menu_data():
"path": root_path,
"icon": appbuilder.app_icon,
"alt": appbuilder.app_name,
"width": appbuilder.app.config["APP_ICON_WIDTH"],
},
"navbar_right": {
"bug_report_url": appbuilder.app.config.get("BUG_REPORT_URL"),