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

Formats values in InputfieldPageAutocomplete and ProcessPageSearch #992

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

Formats values in InputfieldPageAutocomplete and ProcessPageSearch #992

wants to merge 3 commits into from

Conversation

esrch
Copy link

@esrch esrch commented Mar 6, 2015

When displaying dates in an Autocomplete field, the displayed date is a timestamp. The same is true for values coming from ProcessPageSearch (and thus being displayed during a search by an Autocomplete field). This pull request formats these values to make them readable.

@ryancramerdesign
Copy link
Owner

I think we'll run into double entity encoding and some fields with obvious encoded HTML and such, so not sure we can take this approach. Dates are kind of an exception since their unformatted value is pretty useless for any kind of selection purposes. I think some more on how we might best handle it.

@esrch
Copy link
Author

esrch commented Apr 1, 2015

In which cases would we run in double entity encoding for example (not trying to argue, just trying to understand, so that I can maybe help solve the problem, since I really need to be able to do this :-) ).

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