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

Allow templating in sortAscending and sortDescending ARIA strings #86

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ferranmesas
Copy link

ARIA labels for sortable columns are built using raw string concatenation. With this pull request, the same string templating that works in other components (like in pagination: Showing page _PAGE_ of _PAGES_) can be used for both of these strings.

I offer this contribution under the same license the project uses, the MIT license.

@DataTables
Copy link
Collaborator

Think looks like a fantastic addition - thanks! I'm not yet sure if I'll include it in 1.10.13, or the following. Let me get back to you on that - I should be able to confirm next week.

@DataTables
Copy link
Collaborator

Sorry for the delay in getting back to you here!

I feel inclined to include this in the next major update (be that 1.11 or 2.0) rather than in 1.10 and simply do away with the option to not have _HEADER_ in the string. It would simplify the code a little and keep it compact.

Hopefully we aren't too far away from branching to the next major version, so I'll include this then.

Avaruz pushed a commit to Avaruz/DataTablesSrc that referenced this pull request Jan 15, 2024
…Tables.AspNet.Mvc5/Newtonsoft.Json-13.0.1

Bump Newtonsoft.Json from 9.0.1 to 13.0.1 in /src/DataTables.AspNet.Mvc5
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.

2 participants