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(map-chat): disable dummy info window #4707

Merged

Conversation

geminiyellow
Copy link
Contributor

Short Description:

msg.text = html_sanitize(msg.text); return undefined as 'undefined'.

Fixes:

disable just appeared info window

@mention-bot
Copy link

@geminiyellow, thanks for your PR! By analyzing the annotation information on this pull request, we identified @BreezeRo and @solderzzc to be potential reviewers

@geminiyellow geminiyellow force-pushed the fix-disable-dummy-info-window branch from 341564b to cc08aac Compare August 25, 2016 03:32
@@ -320,9 +320,8 @@ function displayMessageOnMap(msg, olat, olong, sessid, icostr, expir, pokenick)
}
});

if (msg.text) {
if (msg.text !== 'undefined') {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

display the location is not that bad...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@solderzzc yep, display the location is not bad, but the infoWindow is meaningless, i dont think human can read it. you can try it by yourself. btw, the same info is output in console.debug, too.
and, if you want a infoWindow, i think there should have a filter.just pop up what you want.

@solderzzc
Copy link
Contributor

solderzzc commented Aug 25, 2016

I've understood what you meant, thanks a lot.
👍

Approved with PullApprove

@solderzzc solderzzc merged commit 0b86fba into PokemonGoF:dev Aug 25, 2016
@solderzzc
Copy link
Contributor

I'd like to have the dummy info back, till we find a better GUI.
Since the box shows how fast the pokemon appear.
Or make a timeout to clean up?

@geminiyellow geminiyellow deleted the fix-disable-dummy-info-window branch August 26, 2016 03:44
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.

3 participants