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

Update [email protected] and [email protected] #1160

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

fel1x-developer
Copy link
Contributor

Screenshot 2024-06-29 113808
Screenshot 2024-06-29 113820
Screenshot 2024-06-29 113833
Screenshot 2024-06-29 113841

@bapt
Copy link
Member

bapt commented Aug 20, 2024

while it look clearly more modern, but I think poudriere ui needs to remind compact and have as many as possible useful information in the main page.

@fel1x-developer
Copy link
Contributor Author

while it look clearly more modern, but I think poudriere ui needs to remind compact and have as many as possible useful information in the main page.

My current plan is to import the latest version of bootstrap and datatables, then add features & rework on the mainpage.

Currently the import of Datatables of 2.x is ready out of tree, for which I will create a PR after this PR is closed.

As I am not familiar with web development using jquery and plain html, I am considering using node.js to rebuild the web app. Using React.js or Vue.js with typescript will make development and maintaince way easier than using jquery & plaint html.

I can start making experiments on that in September, so I hope this PR and the next PR (Datatables 2.x) be merged by the end of this month.

@fel1x-developer
Copy link
Contributor Author

@bdrewery , can you please review this PR?

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.

2 participants