-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[ROADMAP] Update the roadmap #11606
[ROADMAP] Update the roadmap #11606
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,13 @@ | ||
# Roadmap | ||
|
||
The roadmap is a living document, and it is likely that priorities will change, but the list below should give some indication of our plans for the next major release, and for the future. | ||
|
||
Our priorities: | ||
1. **Material Design**. The Google Design team has recently released a new iteration on the Material Design specification, it’s focused on *themability*. This is great news! We will do our best to be up to date. It also confirms that heavily investing in customizability for v1 was a good call. We will continue our efforts. | ||
1. **More utils**. We want to work on new utility components: Spacing, Display, Color, etc. | ||
1. **Performance**. We can’t optimize something we can’t measure, and we don’t currently have a CI performance benchmark, so we will need to build one and start investigating bottlenecks. | ||
1. **Learning materials**. The quality of the documentation is equally as important as the quality of the implementation, and while the reference documentation is comprehensive, we could author a learning tutorial like Next.js has done, or an egghead.io course. | ||
Themes. We want to provide common layouts example to make getting started even easier. We also plan on adding more premium themes. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. '1. **' ? |
||
1. **Documentation**. We want to translate the documentation into Chinese and more languages. Any help is welcomed! | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 'and other languages' |
||
1. **Bundle size**. The library needs to be as small as possible, so we need to work on solutions to further reduce the bundle size, for instance, supporting Preact or Nerv can help. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 'for example, supporting Preact or Nerv.' |
||
1. **Type checking**. We will continue to improve the TypeScript and maybe the Flow coverage of the library. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 'and possibly' |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,9 @@ | ||
/* eslint-disable react/no-danger */ | ||
|
||
import 'isomorphic-fetch'; | ||
import React from 'react'; | ||
import Button from '@material-ui/core/Button'; | ||
import Snackbar from '@material-ui/core/Snackbar'; | ||
import 'isomorphic-fetch'; | ||
|
||
function getLastSeenNotification() { | ||
const seen = document.cookie.replace( | ||
|
@@ -43,11 +43,17 @@ class Notifications extends React.Component { | |
message: {}, | ||
}; | ||
|
||
componentDidMount = async () => { | ||
async componentDidMount() { | ||
this.mounted = true; | ||
|
||
// Prevent search engines from indexing the notification. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
if (/glebot/.test(navigator.userAgent)) { | ||
return; | ||
} | ||
|
||
await getMessages(); | ||
this.handleMessage(); | ||
}; | ||
} | ||
|
||
componentWillUnmout() { | ||
this.mounted = false; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,9 +2,9 @@ import React from 'react'; | |
import PropTypes from 'prop-types'; | ||
import classNames from 'classnames'; | ||
import Button from '@material-ui/core/Button'; | ||
import CheckIcon from '@material-ui/icons/Check'; | ||
import ErrorOutlineIcon from '@material-ui/icons/ErrorOutline'; | ||
import InfoIcon from '@material-ui/icons/InfoOutline'; | ||
import CheckCircleIcon from '@material-ui/icons/CheckCircle'; | ||
import ErrorIcon from '@material-ui/icons/Error'; | ||
import InfoIcon from '@material-ui/icons/Info'; | ||
import CloseIcon from '@material-ui/icons/Close'; | ||
import green from '@material-ui/core/colors/green'; | ||
import amber from '@material-ui/core/colors/amber'; | ||
|
@@ -15,9 +15,9 @@ import WarningIcon from '@material-ui/icons/Warning'; | |
import { withStyles } from '@material-ui/core/styles'; | ||
|
||
const variantIcon = { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Only filled icons for consistency. |
||
success: CheckIcon, | ||
success: CheckCircleIcon, | ||
warning: WarningIcon, | ||
error: ErrorOutlineIcon, | ||
error: ErrorIcon, | ||
info: InfoIcon, | ||
}; | ||
|
||
|
@@ -38,6 +38,7 @@ const styles1 = theme => ({ | |
fontSize: 20, | ||
}, | ||
iconVariant: { | ||
opacity: 0.9, | ||
marginRight: theme.spacing.unit, | ||
}, | ||
message: { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'specification which is focused'
'to keep up to date'
'It also validates that'
'was a good call. We will continue our efforts in this area.'