New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Jlicon migration #7700
Jlicon migration #7700
Commits on Jan 3, 2020
-
-
-
-
-
all of the wrapped icons now get autogenerated by
integrity
also had to revert the cleanup of createIcon's return type. The new way didn't work since some template parameters weren't specified
-
added explicitly declared return type to
createIcon
also added explicit types to the call to `React.forwardRef` within `createIcon`
-
simplified createIcon function by refactoring it into JLIcon class
individual icons are now instances of JLIcon class
-
refactored JLIcon.react from accessor => property
more efficient, since this way the component returned by .react is created only once (as part of the constructor) and cached. Need to be careful with future changes to JLIcon.constructor, as execution order of initializers is now important
-
-
-
all tabbar icons now rendered by blended react/phosphor vdom elements
requires the changes in the phosphorjs/phosphor#437 PR
-
fixed sidebar tab icons; styling still isn't right
currently can't use two different styles for tab icons passed to phosphor. Also in this commit: an experiment in constructing a bare svg react component from a raw string (contained in jlicon.tsx)
-
-
-
-
-
-
human readability formatting pass of filetype icon svg
punted on issue of tabs-vs-spaces for indents
-
-
-
-
-
-
-
-
-
-
-
-
-
simplified JLIcon styling for lumino; moved all tab icons to JLIcon
All filetype icons in tabs and all extension icons in the sidebar are now JLIcons
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
finished annotating all svg icons for themeability
possible I still missed a couple
-
-
-
-
-
minor regressions in search icons and a couple of caret icons
-
-
-
-
-
-
-
-
-
upgraded lumino deps to pull latest advances in virtualdom icon handling
tested the JLIcon stuff, appears to work well
Commits on Jan 4, 2020
Commits on Jan 6, 2020
-
Steven Silvester committed
Jan 6, 2020 -
Add a className to the breadcrump ellipsis
Steven Silvester committedJan 6, 2020 -
Steven Silvester committed
Jan 6, 2020 -
Steven Silvester committed
Jan 6, 2020 -
-
Merge branch 'jlicon_migration' of github.com:telamonian/jupyterlab i…
…nto jlicon_migration
-
-
Steven Silvester committed
Jan 6, 2020 -
-
Steven Silvester committed
Jan 6, 2020 -
Steven Silvester committed
Jan 6, 2020