Fix errors when no default theme is included

jarek-foksa / xel

Xel - Widget toolkit for building native-like Electron and Web apps

https://xel-toolkit.org

Other

679 stars 58 forks source link

Fix errors when no default theme is included #56

Closed depeele closed 6 years ago

depeele commented 6 years ago

Add utils/theme.js : themePath(), themeName()

perform error-tolerant retrieval of the theme path and name, defaulting to node_modules/xel/stylesheets/vanilla.theme.css if no xel theme is included and the theme selector document.querySelector('link[href*=".theme.css"]') returns null;
use these utilities in xel-app.js, xel-codeview.js, and xel-demo.js to avoid errors when no xel theme is included by default;

jarek-foksa commented 6 years ago

Thanks!

I have renamed the utility functions to getThemeName() and getThemePath and I made other adjustments to make your changes consistent with the rest of the codebase.

BTW, if you add new files to the project or make substantial changes to existing files, please add yourself to the @copyright header.