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

New plugin: Copy portal name and links to clipboard #935

Closed
wants to merge 1 commit into from

Conversation

tapionx
Copy link

@tapionx tapionx commented Jan 26, 2015

When you press CTRL+C, this plugin copy to the clipboard the currently
selected portals, with the following syntax:

$portal_name
$portal_intel_link
$portal_gmaps_link

This is very useful for operators, allowing them to send coordinates of a portal to ground agents in a chat.

schermata del 2015-01-26 13 09 43

When you press CTRL+C, this plugin copy to the clipboard the currently
selected portals, with the following syntax:

$portal_name
$portal_intel_link
$portal_gmaps_link

This is very useful for operators, for sending coordinates of a portal to ground agents in a chat.
@tapionx
Copy link
Author

tapionx commented Jan 26, 2015

This little tweak could be joined directly in IITC in my opinion.. I decided to write a plugin to avoid messing up the core IITC code :)
I tried to follow the source code guidelines in the "HACKING" file.

@flamusdiu
Copy link

well, that would work if ctrl+c didn't copy something else at the same time from somewhere else. =)

@tapionx
Copy link
Author

tapionx commented Jan 26, 2015

What do you mean? you must have the IITC browser window active and in foreground..
What did it copy?

@flamusdiu
Copy link

@tapionx I think I didn't explain very well: it would be fine as long as no other functions auto fire copy (if they do then need to fix the race condition for it). I could see getting some odd issues if more then one function would fire on the copy key.

@lapo-luchini
Copy link

In the Trello way you copy what you hover, there is no ambiguity. (unless
the two elements are nested, but that's easily avoided)

Il giorno mar 27 gen 2015 00:34 flamusdiu [email protected] ha
scritto:

@tapionx https://github.com/tapionx I think I didn't explain very well:
it would be fine as long as no other functions auto fire copy (if they do
then need to fix the race condition for it). I could see getting some odd
issues if more then one function would fire on the copy key.


Reply to this email directly or view it on GitHub
#935 (comment)
.

@flamusdiu
Copy link

@lapo-luchini might work. I was just pointing out small problem with it. Either way, either direction would be helpful to operators.

@tapionx
Copy link
Author

tapionx commented Feb 4, 2015

up! :)

@tapionx
Copy link
Author

tapionx commented Mar 19, 2015

UP? :)

@tapionx
Copy link
Author

tapionx commented May 4, 2016

UP !?!?!?!?!!?

// use own namespace for plugin
window.plugin.copyPortalToClipboard = function() {};

window.plugin.copyPortalToClipboard.TrelloClipboard = function() {
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove all references to Trello.

@dingram
Copy link
Contributor

dingram commented Jun 13, 2016

Please also reformat to match the coding style conventions.

Sinnohd added a commit to Sinnohd/ingress-intel-total-conversion-legacy that referenced this pull request Oct 15, 2017
johnd0e pushed a commit to johnd0e/ingress-intel-total-conversion that referenced this pull request Jul 2, 2019
johnd0e pushed a commit to johnd0e/ingress-intel-total-conversion that referenced this pull request Jul 2, 2019
@tapionx
Copy link
Author

tapionx commented Dec 20, 2022

so long, and thanks for all the fish!

@tapionx tapionx closed this Dec 20, 2022
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.

4 participants