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

Issue 1195 - Move Copy to Clipboard to a function to reduce code duplication #1218

Merged
merged 6 commits into from Jan 15, 2020

Conversation

Ruko2010
Copy link
Collaborator

No description provided.

@Ruko2010 Ruko2010 self-assigned this Jan 12, 2020
@Ruko2010
Copy link
Collaborator Author

Ruko2010 commented Jan 12, 2020

@2Abendsegler schaust du bitte nochmal drüber. Ich habe die Funtion angelegt und die 3 Stellen ausgetauscht, ich hoffe ich hab keinen Copy Link vergessen.

@2Abendsegler
Copy link
Owner

Hab heute keine Lust, das Teil muss noch warten. 🙁

@capoaira
Copy link
Collaborator

Ich kenne auch nur die drei Copybuttons.

Copy link
Owner

@2Abendsegler 2Abendsegler left a comment

Choose a reason for hiding this comment

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

Meine Fresse, da muss man ja studiert haben um das zu verstehen. 😂
Das kann ich nicht mal so zwischendurch machen. Ich versuchs morgen oder heute Abend.

Die function gehört eigentlich weiter runter in den Bereich // GClh Functions.

Copy link
Owner

@2Abendsegler 2Abendsegler left a comment

Choose a reason for hiding this comment

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

Die GC Codes funktionieren prima.

Zu den Koordinaten:
Wenn GCTour im Einsatz ist und mit GCTour Koordinaten verschoben wurden dann werden diese nicht berücksichtigt. Meines Erachtens hängt das damit zusammen, dass die Ermittlung der Koordinaten nicht erst beim Klick durchgeführt wird. Die GCTour Daten sind zu diesem Zeitpunkt noch gar nicht vorhanden. Die Ermittlung sollte nach meinem Verständnis immer erst dann erfolgen, wenn der Button betätigt wird, auch für den GC Code, nur macht es dort keinen Unterschied.

(Hier wird auch deutlich dass der Button rechts "Copy Data to Clipboard" auch noch nicht sauber funktioniert. Auch hier darf der Aufbau der Punkte im Dropdown erst stattfinden wenn man mit der Maus darüber geht. Und er muss auch immer wieder erneut stattfinden, weil sich zwischenzeitlich die GCTour Koordinaten geändert haben könnten. Hier werden zwar die richtigen Koordinaten ermittelt, es wird aber nicht bemerkt, dass es keine geänderten Koordinaten mehr gibt und deshalb der Punkt aus dem Dropdown entfernt werden müßte.
Dazu mache ich aber ein eigenes Issue auf. Ich werde mich darum kümmern.)

@Ruko2010
Copy link
Collaborator Author

Ich glaube ich verstehe das Problem. Ich werd mir mal GCTour installieren und dann schauen was genau du meinst.
Aber noch eine dumme Frage. Warum verschiebt man überhaupt Koordinaten per GCTour und nicht über das Standardfeature von GS?

@2Abendsegler
Copy link
Owner

Eigentlich ist es nicht notwendig GCTour zu installieren. Es muss lediglich sichergestellt werden, dass die Koordinaten erst beim Klick ermittelt werden.

Ich vermute die GCTour Koordinaten entstammen noch einer Zeit in denen es mit geänderten Koordinaten im Cache nicht so weit her war. Vermutlich könnte GCTour heute auch mit den geänderten Cache Koordinaten arbeiten. Ich nutze die Koordinaten von GCTour ständig.

@Ruko2010
Copy link
Collaborator Author

Ich hab die Funktion zum Kopieren der Koordinaten wieder raus genommen bei mir. Außerdem ist die ctoc funktion nun an der richtigen stelle im Coding.

Copy link
Owner

@2Abendsegler 2Abendsegler left a comment

Choose a reason for hiding this comment

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

Aus meiner Sicht prima.

Allerdings gibt es scheinbar ne Überschneidung im Coding zu Copy coordinates to clipboard.
Wir können aber trotzdem mergen und ich schieb die Teile nochmal nach.
Soll ich mergen und nachschieben?

@2Abendsegler
Copy link
Owner

Habe die Konflikte noch beseitigt.

@Ruko2010
Copy link
Collaborator Author

Du hättest auch gern mergen können 😃

@Ruko2010 Ruko2010 merged commit 583cd9a into 2Abendsegler:collector Jan 15, 2020
@Ruko2010 Ruko2010 deleted the feature/1195-ctoc branch January 15, 2020 11:43
@2Abendsegler
Copy link
Owner

#1195

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.

None yet

3 participants