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] Disabled Option #23

Merged
merged 3 commits into from
Apr 7, 2022
Merged

[Update] Disabled Option #23

merged 3 commits into from
Apr 7, 2022

Conversation

ItsANoBrainer
Copy link
Contributor

@ItsANoBrainer ItsANoBrainer commented Feb 25, 2022

  • Reworked .js code to work off of event listeners instead of trying to find the button based off every single document click
  • Added support for the "disabled" option parameter, takes a bool, when true you can't click on it
  • Disabled entry has a greyed out background
  • Added underline to header menu entry

image

- Reworked .js code to work off of event listeners instead of trying to find the button based off every single document click
- Added support for the "disabled" option parameter, takes a bool, when true you can't click on it
- Disabled entry has a greyed out background
- Added underline to header menu entry
@BerkieBb
Copy link
Contributor

Did you test everything? Including return values?

@ItsANoBrainer
Copy link
Contributor Author

Yes I've been using this for a month or so.

@xenfovn
Copy link

xenfovn commented Feb 27, 2022

actually, we need it to shorten instead of having to create multiple menus , nice work

- Doesnt even create/render the option. Can be used instead of disable option
@ItsANoBrainer
Copy link
Contributor Author

ItsANoBrainer commented Feb 27, 2022

actually, we need it to shorten instead of having to create multiple menus , nice work

@xenfovn
I've added support for a "hidden" option that doesn't create the entry at all.

It is easier to do this and set it to a dynamically changing bool instead of dynamically creating the table based on that variable.

Copy link
Contributor

@FjamZoo FjamZoo left a comment

Choose a reason for hiding this comment

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

Been using this for some time and i've run into 0 issues. Finally a non breaking change 👏

@GhzGarage GhzGarage merged commit d5253b5 into qbcore-framework:main Apr 7, 2022
@DominonZ
Copy link

DominonZ commented Feb 3, 2023

Am I completely dumb, or am I missing something?

Each time I restart my Dev, the Tracks I've created are gone.
Is this an issue, or am I the issue.

Otherwise the script work as intended.

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.

6 participants