-
Notifications
You must be signed in to change notification settings - Fork 115
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
TextFields are missing the option for a trailing visual icon or text #3218
Comments
Hi, We are planning to implement the trailing_visual feature, particularly for text, as it's something we need. However, we’d like to understand your preferences for how trailing text should be displayed, especially when it’s long. Here are some potential solutions we’ve considered:
We’d appreciate your feedback to ensure the implementation meets expectations and works seamlessly. Best regards, |
Hi! @bsatarnejad thanks for the offer to contribute this feature! Before we go that route, I'd like to confirm that this feature isn't something that already exists (and may have just been hard to find). I'm adding this to our First Responder project board to address that question. @camertron can you please investigate and confirm on this issue whether trailing visuals already exist on |
Hey there 👋 We actually do support a trailing visual clear button, but that's it, unfortunately. Support for text, icons, etc, would be great 😄 |
Hi @lesliecdubs and @camertron |
@bsatarnejad I took a look at the trailing visual slot for the React version of the
Because render(
Primer::Alpha::TextField.new(
trailing_visual: {
# Normally you'd only pass one of these options,
# I'm showing all of them for illustration purposes
icon: :accessibility,
text: "Foobarbaz",
counter: 5,
label: "Hello world"
}
)
) |
UseCase
As a user,
I want to see the unit of a value close to it, ideally inside the text field,
so that I can easily understand the value.
Problem
In the Primer docs, there is the option for trailing icons or trailing text inside text fields described:
Unfortunately, the
trailing_visual
is not implemented yet.The text was updated successfully, but these errors were encountered: