-
Notifications
You must be signed in to change notification settings - Fork 1
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
Single page application #63
Conversation
front/doc/component.md
Outdated
|
||
-------------------------------------------------------------------------------- | ||
|
||
# Redefined Methods |
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.
Maybe change to "Methods to redefine when implemnting" or smth like that
front/doc/component.md
Outdated
|
||
## style | ||
|
||
Returns the CSS content to be rendered inside the component. |
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.
Returns the CSS content to be rendered inside the component. -> Returns the specific CSS content to be rendered inside the component.
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.
very good documentation, makes it easy to review the PR.
let cookies = document.cookie.split(';'); | ||
for (let i = 0; i < cookies.length; i++) { | ||
let c = cookies[i].trim(); | ||
if (c.indexOf(name) === 0) { |
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.
Nothing important, but consider using c.startsWith(name)
as it will avoid iterating through the whole string for no reason
The purpose of this pull request is to remove the old component system and introduce the new single page application.