search

akcdotorg / akc.org

0 stars 0 forks source link

Unformatted chars #250

Closed toshan2014 closed 8 years ago

toshan2014 commented 8 years ago

http://stage-ng.akc.org/dog-breeds/?letter=B

See attachment:

unformatted_chars
stevepessah commented 8 years ago

We had resolved this issue before and it's back again, FYI.

stevepessah commented 8 years ago

Another example can be found here - https://github.com/akcdotorg/akc.org/issues/97

Needs to be updated throughout site.

skt2 commented 8 years ago

In first example, it seems title is copied from somewhere so the dash(-) is some different character. when I changed – to dash( - ) in my local CMS, it solved the problem. National Purebred Dog Day is May 1 – Show your Support National Purebred Dog Day is May 1 - Show your Support If this is the case then we might need to be careful while adding content though I'll keep looking into this but just to make sure that this is not happening due to copy & paste content from somewhere.

screen shot 2016-05-12 at 4 47 44 pm

skt2 commented 8 years ago

Tested Here: http://develop2.akc.org/dog-breeds/ unformatted_chars

skt2 commented 8 years ago

For this to work, I have removed, {exp:akc_apps_path_env:outputCorrectDomainBasedOnEnv} the markup was wrapped inside this and that was causing the issue, I don't see any issue by removing this but @spaulenoff please ask Aspire to QA is throughly if they can find any issue.

skt2 commented 8 years ago

Tested here: http://stage-ng.akc.org/dog-breeds/

aalattili commented 8 years ago

@spaulenoff & @skt2 QA retesting passed on dev2 & stage across browsers (GC 50, FF 45 & IE 11).

skt2 commented 8 years ago

Time Spent: 3h

aalattili commented 8 years ago

@spaulenoff Verified on prod, no article was found with same issue but couldn't find the article used to have it

spaulenoff commented 8 years ago

Verifiedo n Prod