Skip to content
This repository has been archived by the owner on Jan 28, 2023. It is now read-only.

Refactorings, cleanup, etc. #57

Open
2 of 7 tasks
PzYon opened this issue Aug 9, 2018 · 0 comments
Open
2 of 7 tasks

Refactorings, cleanup, etc. #57

PzYon opened this issue Aug 9, 2018 · 0 comments
Labels
pure technical Pure technical, not visible for user, etc.

Comments

@PzYon
Copy link
Owner

PzYon commented Aug 9, 2018

  • ViewItemPage and EditItemPage could both be wrapped by a DoWithItemPage or something
  • Rename common-folder to reusable, e.g. move Markdown.tsx and CodeEditor.tsx there
  • Replace constructors with public readonly state: I..State
  • Create ActionButton or something for anchors (reason: to have a unified "action" component, a11y)
  • Analyze usages of inline function, especially in for/map (reason: performance)
  • Run Audit in Chrome - see if something interesting is printed
  • Consider adding react-hooks eslint plugin
@PzYon PzYon added the pure technical Pure technical, not visible for user, etc. label Aug 9, 2018
@PzYon PzYon changed the title Refactorings Minor refactorings Aug 26, 2019
@PzYon PzYon changed the title Minor refactorings Refactorings, cleanup, etc. Oct 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pure technical Pure technical, not visible for user, etc.
Projects
None yet
Development

No branches or pull requests

1 participant