-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Refactor login page #355
Refactor login page #355
Conversation
- Add 'PasswordLogin' UI component - Add 'LoginPage' wire component which, along with Signup from react SDK, replaces the 'Login' page. - Move UI code (state/props) from ServerConfig which was lobotomoised in the React SDK. Unfinished.
…hough in React like this.
@@ -0,0 +1,199 @@ | |||
/* |
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.
The components this pulls in are from the new src/components/ directory, split by functionality, but this new style non-split component is still part of the views hierarchy, mixed in with all the old, split components? 'views' were supposed to be the UI part of a component: this is a complete component, not a view?
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.
The intention is for this to be a complete component. Based on your comments, would you prefer if it were in the components directory to avoid confusion?
See comments in matrix-org/matrix-react-sdk#31 |
/me wonders why this has been assigned to me |
LGTM'd by Dave irl |
This is the sister PR to matrix-org/matrix-react-sdk#31.
This PR:
ServerConfig
a normal component rather than just the view.CasLogin
and uses the entire component from React SDK.templates/Login
withpages/Login
which wires together UI components with theSignup
logic.