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 settings layout #96

Merged
merged 1 commit into from
May 10, 2021
Merged

Update settings layout #96

merged 1 commit into from
May 10, 2021

Conversation

kam56
Copy link
Contributor

@kam56 kam56 commented May 10, 2021

Complete:

  • Update the overall layout of the settings

  • Add respective pages in a separate folder

Next:

  • Improve the functionality of the tables

Some parts of the code and the layout is not completely finalized.
But since another issue is dependent on this, I commit it now.
So please review it just for the overall look.

Contributes #64

* Update the overall layout of the settings page

* Add respective pages in a separate folder

Next:

* Improve the functionality of the tables

Contributes with #64
@kam56 kam56 added sprint-3 User stories & bugs for TBD on sprint 3 JS Javascript work CSS CSS related task NodeJS NodeJS and ElectronJS related tasks M3 Milestone 3 related issues labels May 10, 2021
@kam56 kam56 requested a review from axe007 May 10, 2021 08:09
@kam56 kam56 self-assigned this May 10, 2021
Copy link
Contributor

@OneMoreOreo OneMoreOreo left a comment

Choose a reason for hiding this comment

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

Very well done! Now let´s add more functionality.

@OneMoreOreo OneMoreOreo merged commit 6198b21 into master May 10, 2021
@kam56 kam56 deleted the settings_layout branch May 15, 2021 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CSS CSS related task JS Javascript work M3 Milestone 3 related issues NodeJS NodeJS and ElectronJS related tasks sprint-3 User stories & bugs for TBD on sprint 3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants