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

Ability to copy text and open urls from the Chat Log GUI #417

Closed
100best opened this issue Aug 25, 2016 · 13 comments
Closed

Ability to copy text and open urls from the Chat Log GUI #417

100best opened this issue Aug 25, 2016 · 13 comments
Labels
enhancement request/proposal to add a new feature
Milestone

Comments

@100best
Copy link

100best commented Aug 25, 2016

I think it would be nice to be able to copy text and make URLs clickable in the Chat Log GUI.

@feikname
Copy link
Collaborator

Note: The workaround for now is to open the game using a terminal window (Command Prompt on Windows) and copy the chat logs from there.

@ghost
Copy link

ghost commented Sep 23, 2016

There are logs storend in %appdata%/openspades (.openspades also i think)

@100best
Copy link
Author

100best commented Sep 23, 2016

For me it is stored in ~/.local/share/openspades.

@VelocityRa VelocityRa added the enhancement request/proposal to add a new feature label Nov 5, 2016
@100best 100best changed the title Copy text from Chat Log Ability to copy text and open urls from the Chat Log GUI Nov 12, 2016
@ghost
Copy link

ghost commented Nov 24, 2016

For some reason ctrl+c and ctrl+v crash the game (?)

@feikname
Copy link
Collaborator

@Tapmemer inside the Chat Box? which version?

@ghost
Copy link

ghost commented Nov 24, 2016

Inside chat box, version 0.0.12B(i know it doesnt help but i dont know the update version)

@feikname
Copy link
Collaborator

feikname commented Nov 24, 2016

@Tapmemer That's weird, mine works fine. For how long have you been having this problem?

If you can confirm that this Copy and Paste Problem persists even after restarting OpenSpades and/or your computer. Please open a new issue here.

On a side note, I will add the Snapshots page to Wiki when I get some free time. So people can test if their issue still exists in the latest development version.

@feikname feikname added this to the 0.1.0 milestone Nov 30, 2016
@yvt yvt modified the milestones: 0.1.1, 0.1.0 Dec 16, 2016
@ghost
Copy link

ghost commented Dec 31, 2016

it happens over every release of openspades

@feikname
Copy link
Collaborator

@Tapmemer Have faith, now development is active again :)

@VelocityRa
Copy link
Collaborator

How is this going to get implemented?

@noway
Copy link
Contributor

noway commented Jan 6, 2017

ability to click urls is somewhat feasible: underscore the piece on hover. get those pieces by parsing using regexes

@feikname
Copy link
Collaborator

feikname commented Jan 6, 2017

Processing the string before rendering it seems the best option.

Maybe implementing it like Minecraft did would be a good idea. When the user clicks, prompt it with 3 options:

  • Copy to clibpboard
  • Open in default browser
  • Cancel

Also, we should only allow http and https to be clicked, for obvious reasons.


EDIT: We would also need to create "Clickable" and "Underlined" text properties probably.

@feikname
Copy link
Collaborator

feikname commented Jan 6, 2017

Perhaps we could make URL's clickable only in a next version too.

Implementing only the copy-and-paste would already help a lot of users, and allow them to paste the URL in their browser, which seems not so bad.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement request/proposal to add a new feature
Projects
None yet
Development

No branches or pull requests

5 participants