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.

Sign up for GitHub

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

Jump to bottom

Details on process-by-name grouping aren't showing both processes #281

Closed
tomwilkie opened this issue Jun 24, 2015 · 3 comments
Closed

Details on process-by-name grouping aren't showing both processes #281

tomwilkie opened this issue Jun 24, 2015 · 3 comments
Assignees
@davkal
Labels
bug Broken end user or developer functionality; not working as the developers intended it

Comments

@tomwilkie
Copy link
Contributor

No description provided.

@tomwilkie tomwilkie added the bug Broken end user or developer functionality; not working as the developers intended it label Jun 24, 2015
@tomwilkie tomwilkie self-assigned this Jun 24, 2015
@tomwilkie tomwilkie mentioned this issue Jun 24, 2015
Merged
@tomwilkie
Copy link
Contributor Author

This was causes by newline (\n) making it into rendered node ids. I've trimmed the newline off the process name to prevent this.

I believe that was another UI issue @davkal; looks like the UI doesn't escape newlines in generate URLs correctly. Will leave this ticket open to discuss that.

@tomwilkie tomwilkie removed their assignment Jun 25, 2015
@davkal
Copy link
Contributor

davkal commented Jun 25, 2015

Why are there newlines in the first place?

@tomwilkie
Copy link
Contributor Author

Oh the file we were reading to get the process name had a newline at its end. Thats fixed now. But I think it unearthed a bug in the UI, so probably worth you investigating.

@tomwilkie tomwilkie mentioned this issue Jul 1, 2015
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davkal davkal

Labels
bug Broken end user or developer functionality; not working as the developers intended it
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tomwilkie @davkal