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.

Sign up for GitHub

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

Jump to bottom

feat: web client #204

Open
wants to merge 40 commits into
base: main
Choose a base branch
from
Open

feat: web client #204

wants to merge 40 commits into from

Conversation

SalathielGenese
Copy link
Collaborator

Work In Progress

What's Inside?

  • Angular-based application for the JS Generator web client
  • Monaco Editor as web editor, wrapped into a component
  • Monaco editor exposed as a custom form control
  • The Popup to configure the rendering of JS
  • Tailwind CSS integration for on-the-fly design, with cross-browser compatibility
  • A menu bar, with the configuration button that calls the options' popup
  • A footer with both authorship to OSS Cameroon and link to GitHub repository
  • The linking of this frontend and backend
  • Responsiveness of the editor on mobile
  • Better looking fonts from Google Fonts
  • FortAwesome + Angular integration
  • Internationalization
  • Accessibility-friendly
  • Server-side rendering

@SalathielGenese
feat: init web artifact as maven module
b9f8b78
@SalathielGenese
feat: init yarn package
f0d55be 
yarn init --yes
@SalathielGenese
feat: init angular application
fb57456 
cd jsgenerator-web
ng new jsgenerator-web --new-project-root . --prefix jsgenerator --package-manager yarn --style sass --routing --strict --force
mv jsgenerator-web/.* .
mv jsgenerator-web/* .
rm jsgenerator-web/
@SalathielGenese
feat: clean up generated content
5a6102e
@SalathielGenese
feat: add UI framework
903b7ea 
yarn add tailwindcss postcss autoprefixer --dev
@SalathielGenese
feat: bootstrap Tailwind default configuration
611e888 
yarn tailwindcss init
@SalathielGenese
feat: configure and test Tailwind integration
6caf637
@SalathielGenese
feat: scaffold popup and test it with configuration popup
5976814
@SalathielGenese
feat: render popup content children
85134df
@SalathielGenese
feat: support popup close with escape key
b0a2872
@SalathielGenese
feat: add configuration form
f4817a6
@SalathielGenese
feat: style configuration form
21f73df
@SalathielGenese
fix: label casing
03672ec
@SalathielGenese
feat: add configuration popup title
a3d848f
@SalathielGenese
feat: style popup title
a38124e
@SalathielGenese
feat: introduce theme color
2ef9ead
@SalathielGenese
feat: improve popup close with escape key
f6ee9a2
@SalathielGenese
feat: add backdrop effect to opened popup
5f56c60
@SalathielGenese
fix: the dialog popup opened/not-opened internal status and update pr…
fc26463 
…opagation
@SalathielGenese
feat: model configuration
9ef0dab
@SalathielGenese
feat: provide default configuration
55a4ff5
@SalathielGenese
feat: inject configuration as our Single sSource of Truth
827fda2
@SalathielGenese
feat: design and integrate the navigation bar, from which we can cont…
b6f24e1 
…rol the configuration popup dealing with configuration state
@SalathielGenese
feat: add footer to the web page
a6fca8a
@SalathielGenese
feat: design general layout flow
a763ec2
@SalathielGenese
feat: add full fledge web editor dependency
3c82ff4 
yarn add monaco-editor
@SalathielGenese
feat: configure monaco-editor
bbf4ddd
@SalathielGenese
feat: implement MonaCoEditorService as a loader for the monaco editor…
2eab1ac 
… assets
@SalathielGenese
feat: test Monaco Editor assets loading
3b46722
@SalathielGenese
feat: scaffold an actual Monaco Editor
773bcbe
@SalathielGenese SalathielGenese self-assigned this Dec 12, 2022
@FanJups
Copy link
Member

FanJups commented Dec 23, 2022

Thanks 💯 for this mvp.

image

FanJups
FanJups approved these changes Dec 31, 2022
View reviewed changes
Copy link
Member

@FanJups FanJups left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks very much

@FanJups
Copy link
Member

FanJups commented Jan 4, 2023

Should I merge or wait until you complete the 14 tasks ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FanJups FanJups FanJups approved these changes

Assignees

@SalathielGenese SalathielGenese

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SalathielGenese @FanJups