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

[✅ASSIGNED] Builder CORE Improvements applied for v1.0.16 #87

Open
app-generator opened this issue Aug 1, 2023 · 1 comment
Open
Assignees
Labels
enhancement New feature or request

Comments

@app-generator
Copy link
Owner

STATUS: ASSIGNED to @umarali


Context

@umarali please add all the functional improvements that need to be done in the CORE of the builder.

PRIO:

  • stability
  • codebase pattern
  • remove all runtime exceptions (JS console)

TY!

@app-generator app-generator changed the title [✅ASSIGNED] Builder CORE Improvements [✅ASSIGNED] Builder CORE Improvements applied for v1.0.16 Aug 1, 2023
@app-generator app-generator added the enhancement New feature or request label Aug 1, 2023
@umarali
Copy link
Collaborator

umarali commented Aug 9, 2023

Some quick suggestions:

  1. Replace magic strings with constants, e.g. currentPageTabs.
  2. Avoid using inline styles.
  3. For better state management, consider libraries like Redux or patterns such as the Context API. Also, to optimize DOM interactions, explore frameworks React, Vue, or Angular could be used.
  4. Modularize the code into distinct units, such as tabManager.ts and componentRenderer.ts, for enhanced maintainability and testing. Similarly, onClick also needs to divided into smaller chunks.
  5. Make use of ES6 features properly.

@app-generator

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

No branches or pull requests

2 participants