Skip to content
This repository has been archived by the owner on May 9, 2023. It is now read-only.

encoding problems in website #37

Closed
mstevens opened this issue Apr 29, 2012 · 8 comments
Closed

encoding problems in website #37

mstevens opened this issue Apr 29, 2012 · 8 comments
Assignees

Comments

@mstevens
Copy link

Looking at the documentation at http://myusuf3.github.com/octogit/ it says:

octogit login &ltusername> &ltpassword> # allows you to store your github authentication data octogit create &ltreponame> 'description' # lets you create the repository both locally and on github octogit issues # lets you see all the related issues in the current repository octogit issues &ltnumber> # lets you see a specific issue with summary octogit issues &ltnumber> close # lets you close an issue

There seem to be some HTML encoding / formatting problems.

@myusuf3
Copy link
Owner

myusuf3 commented Apr 29, 2012

can you provide me with a screenshot? and the browser you are using.

@mstevens
Copy link
Author

On further experiment I've noticed something weird with the HTML escaping. You've got &lt when it should be <

I thought it was all browsers but it may just be links. I can't see how to attach files to a github issue, I'll see if I can find your email address.

@dbr
Copy link
Collaborator

dbr commented Apr 30, 2012

Works fine in Chrome, but w3m (0.5.2) has the same problem:

octogit in w3m

Seem like Chrome is adding the missing ; to &lt etc, e.g this displays fine, even though it probably should not:

$ cat blah.html 
&ltTest&gt

@ghost ghost assigned myusuf3 May 7, 2012
@myusuf3
Copy link
Owner

myusuf3 commented May 7, 2012

so I have updated the site @mstevens is it still rendering poorly?

@mstevens
Copy link
Author

mstevens commented May 7, 2012

It's improved - the angle brackets are now rendering correctly.

I don't think your newlines are rendering correctly, but I see that problem in firefox too - possibly due to using a close br tag rather than a regular one.

@myusuf3
Copy link
Owner

myusuf3 commented May 7, 2012

@mstevens can you please provide a screen capture.

@mstevens
Copy link
Author

mstevens commented May 8, 2012

Hi.

I've taken a screenshot with xlinks2

screenshot

@myusuf3
Copy link
Owner

myusuf3 commented May 22, 2012

I dont know what browser you are using; but I am not going to support it. If you want to submit a pull request fixing this issue you are seeing I more than welcome it. :)

@myusuf3 myusuf3 closed this as completed May 22, 2012
dbr added a commit that referenced this issue Dec 16, 2012
Allows `octogit issues #37` in addition to `octogit issues 37`
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants