Skip to content

Feat: New Select All Permissions Button#5658

Open
itzlinaton wants to merge 9 commits into
pterodactyl:1.0-developfrom
itzlinaton:New-Changes
Open

Feat: New Select All Permissions Button#5658
itzlinaton wants to merge 9 commits into
pterodactyl:1.0-developfrom
itzlinaton:New-Changes

Conversation

@itzlinaton

@itzlinaton itzlinaton commented Jun 17, 2026

Copy link
Copy Markdown

Changes:

  • Added a new button called "Select All" to easily select every permission when inviting a subuser to a server.
{9625515A-6F98-41E5-8357-FC087CABF3D3}

@voidvalue-dev

voidvalue-dev commented Jun 18, 2026

Copy link
Copy Markdown
Contributor
image I’d say you do something like this I don’t think a button is the best solution, given the style of Pterodactyl’s interface.

@itzlinaton

itzlinaton commented Jun 18, 2026

Copy link
Copy Markdown
Author

image I’d say you do something like this I don’t think a button is the best solution, given the style of Pterodactyl’s interface.

That does seem like a much better Idea, I could try. Thank You for the suggestion

@itzlinaton

itzlinaton commented Jun 18, 2026

Copy link
Copy Markdown
Author

@voidvalue-dev Do you like this?

Or do you want it to be wrapped around the permission title box container like in the refrence image you provided? (This would require a bit more code modification)
{8385AE42-1D10-4943-B2F8-9983D3738413}

@voidvalue-dev

voidvalue-dev commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

@voidvalue-dev Do you like this?

Or do you want it to be wrapped around the permission title box container like in the refrence image you provided? (This would require a bit more code modification) {8385AE42-1D10-4943-B2F8-9983D3738413}

I like it the way I have it. The way you’ve got it, it looks very empty to me

@itzlinaton

itzlinaton commented Jun 19, 2026

Copy link
Copy Markdown
Author

I like it the way I have it. The way you’ve got it, it looks very empty to me

So with the permission title box container around it?

@voidvalue-dev

Copy link
Copy Markdown
Contributor

I like it the way I have it. The way you’ve got it, it looks very empty to me

So with the permission title box container around it?

Yes

@itzlinaton

Copy link
Copy Markdown
Author

I like it the way I have it. The way you’ve got it, it looks very empty to me

So with the permission title box container around it?

Yes

Sure, I can try that just let me know if you like it.

@itzlinaton

Copy link
Copy Markdown
Author

Is this fine?
{368C5CD9-05A0-4439-8975-97261CA6DA14}

@voidvalue-dev

Copy link
Copy Markdown
Contributor

Is this fine? {368C5CD9-05A0-4439-8975-97261CA6DA14}

I can see that the description is smaller than the others and uses a different font

@itzlinaton

itzlinaton commented Jun 19, 2026

Copy link
Copy Markdown
Author

I can see that the description is smaller than the others and uses a different font

Yeah It is partially messed up, but can fix it. Besides that it is fine?

@itzlinaton

Copy link
Copy Markdown
Author

Issue solved, @voidvalue-dev
{E74321F0-1215-470D-AE7D-6A086A930C3C}

@voidvalue-dev

Copy link
Copy Markdown
Contributor

I can see that the description is smaller than the others and uses a different font

Yeah It is partially messed up, but can fix it. Besides that it is fine?

Yes, i think

@itzlinaton

Copy link
Copy Markdown
Author

I can see that the description is smaller than the others and uses a different font

Yeah It is partially messed up, but can fix it. Besides that it is fine?

Yes, i think

Okay, cool

*  Made permission button use "`TitledGreyBox`" container instead
* Added a new component (`AllPermissionsButton.tsx`) to make code cleaner
* Made permission button use "TitledGreyBox" container instead
* Added a new component (AllPermissionsButton.tsx) to make code cleaner
@voidvalue-dev

voidvalue-dev commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

I can see that the description is smaller than the others and uses a different font

Yeah It is partially messed up, but can fix it. Besides that it is fine?

Yes, i think

Okay, cool

image

EditSubUserModal.tsx
EditSubuserModal.tsx (original name)

Why did you change the name? Have you tried this in your dashboard?

@itzlinaton

Copy link
Copy Markdown
Author

I can see that the description is smaller than the others and uses a different font

Yeah It is partially messed up, but can fix it. Besides that it is fine?

Yes, i think

Okay, cool

image [EditSubUserModal.tsx](https://github.com//pull/5658/changes#diff-79a72267fd021a5a09c1576d3b74c12b4bdeb6087e603b6da97c3186a26116ca) [EditSubuserModal.tsx](https://github.com//pull/5658/changes#diff-5316c13ce1d16787bee68d3f60ca6782daf30d3ac1b0978c8c7dbfd36743cbe7) (original name)

Why did you change the name? Have you tried this in your dashboard?

I accidentally changed the name, oops! Also yeah I did test it but with the correct name.

@voidvalue-dev

voidvalue-dev commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Let me know when you've fixed it
And update the PR photo

@itzlinaton

Copy link
Copy Markdown
Author

Let me know when you've fixed it And update the PR photo

Should be solved, also thanks for the reminder.

@voidvalue-dev

voidvalue-dev commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Let me know when you've fixed it And update the PR photo

Should be solved, also thanks for the reminder.

image

You did it the wrong way round; the one you deleted is the original

@voidvalue-dev

voidvalue-dev commented Jun 21, 2026

Copy link
Copy Markdown
Contributor
image

Why is there a sort of padding/margin in the description?

@voidvalue-dev

Copy link
Copy Markdown
Contributor

And I can’t see that AllPermissionsButton.tsx is being imported or used

@itzlinaton

Copy link
Copy Markdown
Author

You did it the wrong way round; the one you deleted is the original

Thanks for pointing out the mistake, I will try solve it

Why is there a sort of padding/margin in the description?

I am unsure, it just turned out like that when I styled it, I will have to look into fixing it

And I can’t see that AllPermissionsButton.tsx is being imported or used

That is because I used the old file unfortunately, that was my mistake

@itzlinaton

Copy link
Copy Markdown
Author

Issue has been solved I think
{61FBFA1C-91AB-4E37-96B8-373EB8B72BB8}

@voidvalue-dev

Copy link
Copy Markdown
Contributor

Issue has been solved I think {61FBFA1C-91AB-4E37-96B8-373EB8B72BB8}

image

There’s more space here than above; apart from that, everything looks fine to me.

@itzlinaton

Copy link
Copy Markdown
Author

There’s more space here than above; apart from that, everything looks fine to me.

Issue has been solved
{9A96C354-E48A-4301-80B6-861B776804FE}

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