Skip to content
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

Frontend Framework Rewrite (Mithril) #2126

Closed
wants to merge 150 commits into from

Conversation

dsevillamartin
Copy link
Member

Closes #1821.

See #1950 for the library changes before Mithril.

…oute code

TODO: Fix "SyntaxError: '> li:not(.Dropdown-header)' is not a valid selector" when hovering search results in navbar
The property is 'params' instead of 'body' or 'data'
askvortsov1 and others added 23 commits May 3, 2020 09:26
Now both extend src/common/components/Page.tsx
Admin component is the exact same as common
Forum component extends common & adds one line
Only the state has to be saved to app.cache.discussionList
Cleans up the component class
…anager

app.alerts.show() now can take plain object OR AlertState instance - will return AlertState.prototype.key
app.alerts.dismiss() takes AlertState or state key (returned by show())
@askvortsov1 askvortsov1 mentioned this pull request May 9, 2020
65 tasks
@dsevillamartin dsevillamartin mentioned this pull request Aug 5, 2020
@dsevillamartin
Copy link
Member Author

See #2255.

Some code from this PR will be used in the new PR (work has been mostly done already).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Frontend Framework Rewrite
2 participants