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

feat: add clan tags #463

Merged
merged 23 commits into from
Aug 24, 2024
Merged

feat: add clan tags #463

merged 23 commits into from
Aug 24, 2024

Conversation

favna
Copy link
Member

@favna favna commented Aug 24, 2024

Clan tags support both dark and light themes, and also normal and compact modes.
Also updated timestamps component to ensure no wrapping as happens with discord.

mezotv and others added 22 commits August 20, 2024 14:30
Clan tags support both dark and light themes, and also normal and compact modes.
Also updated timestamps component to ensure no wrapping as happens with discord.
fix: clan tags working as expected
This was already renamed on my pc github must have eaten it up.
* feat: adding new "error" message

I was using the modals recently and I saw that there is one more message in a specific circumstance so I added that message, added new attributes to the class and repositioned the messages

* add: function to show new "error" message

Added a function to show the new "error" message

* Update DiscordModal.ts

* Update DiscordInputText.ts

* fix: Change "minimum characters" message display

Here I added a function to make the "error" message of minimal characters "disappear" when closing the modal

* fix: Fixing some verification errors and adding a listener

I added an extra check on line 410, because as it was, it didn't show the minimum character message in short inputs, and I added a listener so that every time the modal is closed the minimum characters "error" disappears

* fix: some small fixes for validations

* fix: change ! To ?

* fix: various fixes

- ensured validation cases work properly
- ensure warnings properly get removed on closing

* Fix: Solving Value Not Returning to Default

* fix: remove unnecessary type

* fix: Closing unclosed tag and fixing indentation

* fix: remove accidental console.log

* Fix: Fixed issue of looping the default value

* Update DiscordInputText.ts

* add: Adding an example of a default value in the modal

* Fix: Adding More Verification

Adding minimum size verification, now only the minimum size message will appear in some cases the same as discord

- When the field is required
- When the value of the field is greater than 1 character and less than the minimum 

I added this check because before there was none and it always showed the message even if the inputtext was not necessary

* fix: Changing a atributte of class and add news arguments

Here i changed the atribute "hasWarning" to public for use in modal class (more for polishing) e adicionei mais algumas verificações especificas na contagem de caracters (again to polishing)

* fix: changing variable in verification

Here a changed the variable "defaultValue" for the "value" in line 352 and 353,in view of that the variable "value" recieves the variable "defaultValue"

* fix: ensure body scroll lock also works on ios safari

---------

Co-authored-by: Jeroen Claassens <support@favware.tech>
* feat: adding function if you click an option

I added an argument to the class to execute a function by clicking on the selection if the person wants

(I saw this in modal)

* fix: orrecting Indentation
Discord doesnt have wrapping on timestamps, so this ensures no wrapping.

Co-authored-by: Jeroen Claassens <support@favware.tech>
* refactor: update getGlobalEmojiUrl to parse Discord emoji syntax and inline emojis in titles/fields

* Add animated to regex directly

Co-authored-by: Jeroen Claassens <jeroen.claassens@live.nl>

* chore: really ignore ignored worsd

---------

Co-authored-by: Jeroen Claassens <jeroen.claassens@live.nl>
Co-authored-by: Jeroen Claassens <support@favware.tech>
@favna favna marked this pull request as ready for review August 24, 2024 13:12
@favna favna merged commit afe7ee9 into main Aug 24, 2024
6 checks passed
@favna favna deleted the feat/add-clan-tags branch August 24, 2024 13:32
@favna favna mentioned this pull request Aug 24, 2024
6 tasks
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.

5 participants