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

Expanding the modern look to rest of the interface #1729

Closed

Conversation

Gontier-Julien
Copy link
Contributor

Signed-off-by: Gontier Julien [email protected]

By submitting this pull request, I confirm the following: {please fill any appropriate checkboxes, e.g: [X]}

{Please ensure that your pull request is for the 'devel' branch!}

  • I have read and understood the contributors guide, as well as this entire template.
  • I have made only one major change in my proposed changes.
  • I have commented my proposed changes within the code.
  • I have tested my proposed changes.
  • I am willing to help maintain this change if there are issues with it later.
  • I give this submission freely and claim no ownership.
  • It is compatible with the EUPL 1.2 license
  • I have squashed any insignificant commits. (git rebase)
  • I have Signed Off all commits. (git commit --signoff)

What does this PR aim to accomplish?:

It expand the modern look to the rest of the interface.

How does this PR accomplish the above?:

-Change .nav-tabs-custom from border-radius:3px to border-radius:10px !important.
-Change .nav-tabs-custom > .nav-tabs from border-top-left-radius:3px and border-top-right-radius:3pxto border-top-left-radius: 10px !important and border-top-right-radius: 10px !important
-Change .nav-tabs-custom > .tab-content and .box-footer from border-bottom-left-radius:3px and border-bottom-right-radius:3pxto border-bottom-left-radius: 10px !important and border-bottom-right-radius: 10px !important

What documentation changes (if any) are needed to support this PR?:

None

@DL6ER
Copy link
Member

DL6ER commented Feb 3, 2021

Do you have a screenshot showing your changes?

@DL6ER
Copy link
Member

DL6ER commented Feb 3, 2021

-.nav-tabs-custom > .tab-content { 
+.nav-tabs-custom > .tab-content {
-}
\ No newline at end of file
+}

should fix the linting errors

@dschaper
Copy link
Member

dschaper commented Feb 3, 2021

It's way too rounded for me.

@Gontier-Julien
Copy link
Contributor Author

It up for debate and personal taste after this so maybe play around a bit and tell with you think is the sweet spot ^^

Signed-off-by: Gontier Julien <[email protected]>
@dschaper
Copy link
Member

dschaper commented Feb 3, 2021

The template that I have in mind for the next iteration uses .25rem for the radius on buttons.

@dschaper
Copy link
Member

dschaper commented Feb 3, 2021

@Gontier-Julien
Copy link
Contributor Author

@dschaper you're in the wrong pull request for the button and maybe look with what @PromoFaux proposed.

@dschaper
Copy link
Member

dschaper commented Feb 3, 2021

I'm talking with him right now actually.

@Gontier-Julien
Copy link
Contributor Author

@dschaper no problem.
But with .25rem it while not change much (if at all) compared of the current design of the button.

@dschaper
Copy link
Member

dschaper commented Feb 3, 2021

Yeah, I know.

@PromoFaux
Copy link
Member

I should have better reviewed the rounded corners in the last few PRs (or at least discussed it with the rest of the team)

original:
image

10px:
image

5px:
image

.25rem
image

Now, I know this PR is discussing a different part of the interface, but I think this serves as a good a place as any to discuss, with that said my personal preference here is 5px

@dschaper
Copy link
Member

dschaper commented Feb 3, 2021

I'm not going to pick this as a hill for me, but I'd ditch the px as a value, and even then 5px looks too rounded still.

Maybe something like the top widgets on https://vue.architectui.com/architectui-vue-pro/#/dashboards/management

or

https://vue.architectui.com/architectui-vue-pro/#/dashboards/project-management

@Gontier-Julien
Copy link
Contributor Author

@dschaper just a quick question, what wrong with making some part of the interface a little rounded ?
I know some people don't like rounded interface and it totally fine.
But maybe can we make a option for people that like it that way ?
It all personal taste after all.

@dschaper
Copy link
Member

dschaper commented Feb 3, 2021

Quick Answer, I don't like the look. And yes it's personal.

I don't think adding more options to decide if you want to change the radius of the elements is useful, that's going to be a lot of work for a web interface that is planned on being replaced.

@Gontier-Julien
Copy link
Contributor Author

Oh that cool when the new interface while be here ?
And yeah i totally understand this problem ^^

@PromoFaux
Copy link
Member

Oh that cool when the new interface while be here ?

Soon™ is about as good an answer as you'll get :) There is some work going on in this branch for the 6.0 release (which will feature a new api). Beyond that, we're toying with the idea of a total framework change (see linked ui framework above). But no timelines are really set in stone

For this one after a bit of internal discussion - can we pare the roundedness of the corners back to .25rem instead of 10px, please?

@Gontier-Julien
Copy link
Contributor Author

Exited to see this then ^^
Maybe i close those two pull request and you cloud just revert back this pull request ?
#1691

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.

4 participants