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

fix: prevent timestamp wrapping #462

Merged
merged 2 commits into from
Aug 24, 2024
Merged

fix: prevent timestamp wrapping #462

merged 2 commits into from
Aug 24, 2024

Conversation

dpaulos6
Copy link
Contributor

Discord doesnt have wrapping on timestamps, so this ensures no wrapping.

Discord doesnt have wrapping on timestamps, so this ensures no wrapping.
@dpaulos6 dpaulos6 requested a review from favna as a code owner August 22, 2024 10:14
@favna favna merged commit 7ce20a7 into skyra-project:main Aug 24, 2024
6 checks passed
favna added a commit that referenced this pull request Aug 24, 2024
Discord doesnt have wrapping on timestamps, so this ensures no wrapping.

Co-authored-by: Jeroen Claassens <support@favware.tech>
favna added a commit that referenced this pull request Aug 24, 2024
* feat(markdown): add subtext

* feat(clan-tags): add clan tag base

* fix: remove old subtext code

* feat(clan-tag): add to reply WIP

* fix: clan tags working as expected

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: removing timestamp update

* fix: add clan icon files

* fix: add clan map

* fix: make dynamic clan icon work

* fix: image not showing

* fix: finalize icons and add examples

* fix: small log cleanup

* fix: rename heart.ts

This was already renamed on my pc github must have eaten it up.

* add: New arguments in the modal (#453)

* 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 (#461)

* 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

* fix: prevent timestamp wrapping (#462)

Discord doesnt have wrapping on timestamps, so this ensures no wrapping.

Co-authored-by: Jeroen Claassens <support@favware.tech>

* fix: update the ephemeral eye icon (#460)

* refactor: better emoji parsing (#458)

* 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>

* chore: fix merge

* refactor: cleanup and fixes

---------

Co-authored-by: Dominik Koch <dominik@koch-bautechnik.de>
Co-authored-by: dpaulos6 <itzframepvp@outlook.com>
Co-authored-by: Marciel404 <48138111+eumarciel404@users.noreply.github.com>
Co-authored-by: Digital <github@crni.xyz>
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