Change radius to 10px for a more modern look.

[Gontier-Julien]

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 make the interface look a bit more modern not much but improve the overall visual experience.

How does this PR accomplish the above?:

-Change .small-box and .box from border-radius:3px to border-radius:10px.

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

None

[DL6ER]

Can this be realized without changes in vendor directories (like style/vendor)?

The reason I'm asking is basically two-fold:

• Due to the licenses of the contained third-party software, changes often have to be submitted upstream
• Ont he next update of the component, we will have forgotten about this change and simply replace this file with its updated version - undoing your changes unintentionally.

Overwrites are added in style/pi-hole.css, typically where they are guaranteed to stay in our open license and won't be overwritten on component upgrades.

This may also apply to your other PRs, I haven't had the time to look at them so far.

[Gontier-Julien]

How right sorry about that i while fix this today, i didn't pay attention to this ^^'
Do you want me to make a new pull request to this ?

[Gontier-Julien]

Wrong button*

[DL6ER]

No worries, you can either reuse this PR by undoing your first commit and making a new one + force-push. Or just add the change in a second commit which we can then squash-merge. The latter is probably the easiest.

[Gontier-Julien]

Okay no problems i will reuse this PR request and remove this commit and make a new one.

[Gontier-Julien]

Sorry about this inconvenience ^^' but now it fixed ^^

adressed

[pralor-bot]

This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/pi-hole-core-v5-2-3-web-v5-3-and-ftl-v5-4-released/43009/1

[dschaper]

@Gontier-Julien What is your background for front end? Are you familiar with any frameworks?

[Gontier-Julien]

@dschaper i currently don't "have" a background in this, but i've had worked for two small enterprise where i've done there interface and helped the dev team. I'm still a student at the moment.
And no i don't have any experience in frameworks, but i can still learn.

[dschaper]

I'm very happy to have you contribute and learn as you go, so please don't let that be a barrier to your continuing to contribute. We don't ask for resumes or CVs to contribute, your work speaks for you in that regard.

Front-end design is very much a personal preference for a lot of things but we do come to a consensus almost all of the time.