Skip to content

Conversation

stoneage-mta
Copy link
Contributor

I will send another PR to do the same/similar to admin2 in near future.

image

@xLive
Copy link
Member

xLive commented Jun 9, 2021

I've an idea for the design, something like this:
image

What do you think?

@stoneage-mta
Copy link
Contributor Author

Looks good and I can apply something similar, but first lets wait for other opinions.

@Allerek
Copy link
Contributor

Allerek commented Jun 10, 2021

I've an idea for the design, something like this:
image

What do you think?

Thats cool, simple and easy to understand, even newbies will understand that design.

@Dutchman101
Copy link
Member

I've an idea for the design, something like this:
image

What do you think?

I like this design more than the current one in PR, just add something on the left side to clarify those are the available groups (to assign) and something on the right to clarify those are the current groups for that user

@stoneage-mta stoneage-mta marked this pull request as draft June 11, 2021 18:12
@stoneage-mta
Copy link
Contributor Author

bruhhh looks like i just ruined my branch

@stoneage-mta
Copy link
Contributor Author

I think I finally managed to fix it. Sorry for the useless commits, let me know if you think is better to create a new PR, I still learning this thing called git.

Anyways, this is the new design:
image
Please comment below if you have any suggestions.

@stoneage-mta stoneage-mta marked this pull request as ready for review July 4, 2021 13:01
@Dutchman101 Dutchman101 self-requested a review August 15, 2021 20:38
@Dutchman101 Dutchman101 merged commit fe59ab1 into multitheftauto:master Aug 15, 2021
@patrikjuvonen patrikjuvonen added this to the 1.5.9 milestone Apr 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants