Opencaching Community

Die Plattform opencaching.de => Entwicklung => Thema gestartet von: flopp am 07. April 2013, 12:23:38

Titel: Shortlinks
Beitrag von: flopp am 07. April 2013, 12:23:38
Wir haben ja so schöne Shortlinks: http://opencaching.de/OCF4B9

Diese kommen aber nirgends vor:

Hier könnte man überall den schönen Shortlink verwenden.
Titel: Re: Shortlinks
Beitrag von: Schrottie am 07. April 2013, 12:27:24
Ja, das wäre sicherlich sinnvoll.
Titel: Re: Shortlinks
Beitrag von: flopp am 07. April 2013, 12:45:06
Ich denke ich werde das übernehmen...
Titel: Re: Shortlinks
Beitrag von: mic@ am 07. April 2013, 12:51:01
Zitat von: flopp
Ich denke ich werde das übernehmen...

Prima!
Mich hat schon immer gewundert, warum wir ein und desnelben Cache immer auf unterschiedliche Art ansprechen:
http://www.opencaching.de/viewcache.php?wp=OCF4BA
http://www.opencaching.de/viewcache.php?cacheid=165739

Und Deine Lösung, also http://opencaching.de/OCF4BA
ist klein aber fein!
Titel: Re: Shortlinks
Beitrag von: Schrottie am 07. April 2013, 12:58:02
Und Deine Lösung, also http://opencaching.de/OCF4BA
ist klein aber fein!

Und vor allem schon eine ganze Weile lang möglich. :)
Titel: Re: Shortlinks
Beitrag von: flopp am 07. April 2013, 18:18:32
Pull-Request ist raus ;)

https://github.com/OpencachingDeutschland/oc-server3/pull/62
Titel: Re: Shortlinks
Beitrag von: following am 07. April 2013, 23:17:50
Ich habe es zum Ausprobieren auf den Testserver (http://wiki.opencaching.de/index.php/Entwicklung/Betatest) gelegt:

- Kurz-URL in Benachrichtigungen über neue Caches
- Kurz-URL in Benachrichtigungen über neue Logs
- Kurz-URL in Cachelistings

An der Einbindung in die Cachelistings hab ich noch was zu meckern: :)

- Das Listing verlinkt verwirrenderweise auf sich selbst.

- Es funktioniert nur auf dem www.opencaching.de-Server, aber nicht auf dem Testsystem oder anderen Installationen der OC-Software, da die Adresse http://opencaching.de fest vorgegeben ist.  Stattdessen sollte die Kurz-URL (optional) konfigurierbar gemacht werden.*

- Es belegt immer eine ganze Bildschirmzeile, obwohl es nur sehr selten benötigt wird. An anderen Stellen ist noch jede Menge Platz.

Der erste und dritte Punkt ließen sich in einem erschlagen, indem man wie bei GC einen Link hinter den OC-Code setzt, der die kopierbare Adresse einblendet, z.B. "[Kurz-URL]".

* Die Benachrichtungstexte für neue Caches scheinen auch noch nicht konfigurierbar zu sein, vor allem scheint dort die Übersetzung zu fehlen ...? Schicken wir etwa den Nutzern von opencaching.it und opencachingspain.es deutsche Benachrichtigungsmails?!
Titel: Re: Shortlinks
Beitrag von: flopp am 08. April 2013, 23:52:56
Zur Server-Konfigurierbarkeit:
Korrekt, das sollte man tun. Irgendwo in htdocs/config2/settings-dist.inc.php? Als Default/Fallback könnte man ja '$BASEURL + "/viewcache.php?wp={wp_oc}"' nehmen - ist dann zwar nicht "short", funktioniert aber auf jedem Fall.

Zur Lokalisierung der Mails:
So wie es aussieht, bekommen die Italiener und Spanier deutsche Mails, oder? Ich finde jedenfalls keine Lokalisierung... Im Übrigen bekommen sie auch deutsche Blog-Beiträge ;)

Zur "Platzverschwendung", Selbstverlinkung:
Man könnte es tatsächlich wie bei GC machen. Allerdings habe ich dann die Befürchtung, dass dieses Feature komplett übersehen wird.
Titel: Re: Shortlinks
Beitrag von: flopp am 09. April 2013, 07:17:31
Ach so: fehlt auf dem Testserver die Aktualisierung von util/watchlist/run-watch.php? In den watch-Mails wird nämlich das wp_oc Token nicht ersetzt...
Titel: Re: Shortlinks
Beitrag von: following am 09. April 2013, 14:16:57
Zur Server-Konfigurierbarkeit:
Korrekt, das sollte man tun. Irgendwo in htdocs/config2/settings-dist.inc.php? Als Default/Fallback könnte man ja '$BASEURL + "/viewcache.php?wp={wp_oc}"' nehmen - ist dann zwar nicht "short", funktioniert aber auf jedem Fall.

Jup. Die Scripte in util/... verwenden allerdings noch die alte lib/settings.inc.php. Dort wird die dist-Datei nicht included sondern dient nur als Kopiervorlage (nur auf dem Produktivsystem - für die Entwicklersysteme gibt es die Includedatei settings-dev.inc.php).  D.h. zusamen:

* config2/settings-dist.inc.php für die Listinganzeige
* lib/settings-dist.inc.php als Kopiervorlage
* lib/settings-dev.inc.php für die Entwickler
* Produktivsystem: lib/settings.inc.php

Zitat
Zur "Platzverschwendung", Selbstverlinkung:
Man könnte es tatsächlich wie bei GC machen. Allerdings habe ich dann die Befürchtung, dass dieses Feature komplett übersehen wird.

Hmja. Es bräuchte ein Icon das nicht so leicht übersehen wird (wie das Dreieck auf GC). Auf der Karte ist es der gelbe Stern - allerdings könnte der im Listing als Empfehlungsstern missverstanden werden. Ich denke es sollte aber auf beiden Seiten das gleiche Symbol sein. Vielleicht ein größers Link-Icon (das übliche Kettenglieder-Symbol)?

Oder alternativ die Kurz-URL einfach irgendwo rechts in einem vorhandenen Freiraum hinschreiben, ohne Verlinkung - man könnte sie dann markieren und kopieren.

Die Links auf Mapquest & Google Maps unter der Karte sind z.B. überflüssig, weil beides in der Opencaching.de-Karte enthalten ist. Man könnte das ganze Kartenlinkzeug ersetzen durch einen Link "große Karte", wodurch vertikal Platz frei wird. Dann die Kurz-URL entweder darunter einblenden, oder die Karte nach unten schieben und den Kurzlink darüber setzen.

Ach so: fehlt auf dem Testserver die Aktualisierung von util/watchlist/run-watch.php? In den watch-Mails wird nämlich das wp_oc Token nicht ersetzt...

Die run-watch.php auf dem Testsystem ist auf dem Stand von deinem Commit 86055fe.
Titel: Re: Shortlinks
Beitrag von: following am 09. April 2013, 15:03:41
Dann die Kurz-URL entweder darunter einblenden, oder die Karte nach unten schieben und den Kurzlink darüber setzen.

Braucht dann auch kein Label denke ich, die URL an sich ist selbsterklärend.
Titel: Re: Shortlinks
Beitrag von: flopp am 09. April 2013, 15:06:57
Ich werde das mal ausprobieren. Irgendeine ansprechende Lösung wird es geben...
Titel: Re: Shortlinks
Beitrag von: following am 10. April 2013, 01:49:01
Opencaching.pl hat die Idee mit den Shortlinks heute übernommen:

http://code.google.com/p/opencaching-pl/source/detail?r=4759
Titel: Re: Shortlinks
Beitrag von: following am 11. April 2013, 16:52:33
Ach so: fehlt auf dem Testserver die Aktualisierung von util/watchlist/run-watch.php? In den watch-Mails wird nämlich das wp_oc Token nicht ersetzt...

Bei mir hat es funktioniert, in der Logbenachrichtigung war die Kurz-URL drin.

Wenn noch jemand die Neue-Cache-Benachrichtigungen geprüft hat, kann ich diesen Teil freigeben.
Titel: Re: Shortlinks
Beitrag von: flopp am 11. April 2013, 17:10:58
Wenn noch jemand die Neue-Cache-Benachrichtigungen geprüft hat, kann ich diesen Teil freigeben.

Geht beides:

Zitat
Hallo flopptest,

soeben wurde in deinem Suchradius ein neuer Cache gemeldet:
---
flopptest hat am 11.04.2013 den Cache "Ich bin ein Notification-Test!!!" versteckt.
Entfernung: 2.2 km Richtung Nordost
Typ: Mathe-/Physikcache
Größe: extrem groß

Link: http://opencaching.de/OC9E93
---

Zitat
Hallo flopptest,

Neue Logeinträge deiner Caches:
---
11.04.2013 flopptest hat einen Logeintrag für deinen Cache "Ich bin ein Notification-Test!!!" gemacht.
http://opencaching.de/OC9E93

        Na, ist die URL richtig gesetzt?
---
Titel: Re: Shortlinks
Beitrag von: flopp am 11. April 2013, 17:19:28
Die Modifikation der Listings lassen wir erst mal weg, bis ich mich um eine kompaktere Lösung gekümmert habe?
Titel: Re: Shortlinks
Beitrag von: following am 11. April 2013, 17:37:16
Ja, das fände ich besser. Der Rest ist nun online.
Titel: Re: Shortlinks
Beitrag von: flopp am 11. April 2013, 18:07:36
Ja, das fände ich besser. Der Rest ist nun online.

Funktioniert prima ;)
Titel: Re: Shortlinks
Beitrag von: Schrottie am 11. April 2013, 18:10:46
Jepp. :-)
Titel: Re: Shortlinks
Beitrag von: flopp am 11. April 2013, 20:48:09
Wie wäre denn folgende Modifikation (Änderungen: Shortlink hinter Wegpunkt, gekürzte Kartenlinks, neuer Button-Stil)...
Titel: Re: Shortlinks
Beitrag von: Schrottie am 11. April 2013, 20:50:02
Nice! Kriegst Du das so hin, das es ähnlich wie bspw. bei bit.ly sofort in die Zwischenablage kopiert wird, wenn man da draufklickt?
Titel: Re: Shortlinks
Beitrag von: following am 11. April 2013, 20:58:54
Wenn man im Firefox mit vergrößerter Schrift arbeitet (z.B. weil man etwas schlechtere Augen hat...), passt es dort nicht hin. Bei mir sieht es z.B. so aus:
Titel: Re: Shortlinks
Beitrag von: flopp am 11. April 2013, 21:02:25
Wenn man im Firefox mit vergrößerter Schrift arbeitet (z.B. weil man etwas schlechtere Augen hat...), passt es dort nicht hin. Bei mir sieht es z.B. so aus:

Hm, ok. Das ist ein Problem.
Was hälst du von den optimierten Kartenlinks (ein Klick auf die kleine Karte führt auch zur großen!) und den neuen Buttons?
Titel: Re: Shortlinks
Beitrag von: following am 11. April 2013, 21:10:26
Was hälst du von den optimierten Kartenlinks (ein Klick auf die kleine Karte führt auch zur großen!) und den neuen Buttons?

Oops, ich hatte nur auf den Shortlink geachtet. Die Karte mit dem Link finde ich gut, und die querlaufende Buttonleiste schön übersichtlich und platzsparend (wobei Buttons mit variabler Breite günstiger sein könnten: Wenn man die Schrift etwas vergößert, passt das "An GPS-Gerät senden" im Firefox wieder nicht rein ...)

Das bringt frischen Wind ins Cachelisting, finde ich toll. :)
Titel: Re: Shortlinks
Beitrag von: flopp am 11. April 2013, 21:31:06
OK. Im Prinzip könnte man diese Buttons doch überall einsetzen...
Titel: Re: Shortlinks
Beitrag von: Slini11 am 11. April 2013, 21:38:59
Ich find die neuen kleinen blauen Buttons auch schöner uns übersichtlicher als die alten.
Das gibt dem Listing wirklich mehr frische...
Titel: Re: Shortlinks
Beitrag von: following am 11. April 2013, 21:46:00
OK. Im Prinzip könnte man diese Buttons doch überall einsetzen...

Jup. Ich dachte eigentlich, ich hätte die schon überall eingesetzt außer bei den Listing-Funktionen (9cad450 (https://github.com/OpencachingDeutschland/oc-server3/commit/9cad4504d5b285d0221dc8c56f6b3cde08fc901f)), aber ein paar hab ich wohl übersehen.
Titel: Re: Shortlinks
Beitrag von: flopp am 12. April 2013, 18:32:50
Ein neuer Vorschlag:

* Karte in richtiger Größe (das staticmap-Bild ist 200px groß, wird aber in der aktuellen OC-Version auf 185px beschnitten. Dadurch ist der Copyrighthinweis nur halb sichtbar. Ist das Absicht???)
* Druck-Buttons kombiniert in einem Dropbox wie beim Download
* Buttons mit variabler Größe

Die Buttons könnte man vielleicht noch weiter hochbewegen, so dass sie direkt unter dem Text sind.

Für Peter gibt es zwei Screenshot-Versionen mit Firefox: (1) mit normaler Text-Skalierung und (2) mit lächerlich-großer Text-Skalierung  ;)

Titel: Re: Shortlinks
Beitrag von: bohrsty am 12. April 2013, 18:41:49
das finde ich eine sehr gute moeglichkeit, gefaellt mir!
Titel: Re: Shortlinks
Beitrag von: following am 12. April 2013, 19:06:39
Oh ja, die zusammengefassten Druckbuttons sind eine tolle Idee!

Die Short-URL in dieser Form finde ich immer noch verwirrend. Das ist ein Link, der zum Draufklicken einlädt und auf sich selbst verlinkt.  ???

Wir wär's denn hiermit:
Titel: Re: Shortlinks
Beitrag von: flopp am 12. April 2013, 19:12:47
Dein Vorschlag gefällt mir!
Titel: Re: Shortlinks
Beitrag von: Schrottie am 12. April 2013, 19:21:31
Mir ebenfalls. :-)
Titel: Re: Shortlinks
Beitrag von: flopp am 13. April 2013, 22:43:29
Ich habe das alles mal vorläufig implementiert: https://github.com/flopp/oc-server3/tree/new-listing-header

Da aber mein virtueller OC-Server nicht richtig läuft, fehlt noch ein Praxistest...
Titel: Re: Shortlinks
Beitrag von: following am 14. April 2013, 20:29:42
Ich hab's noch ein wenig aufgebohrt, übersetzt, den Kartenlink korrigiert und auf den Testerver (http://wiki.opencaching.de/index.php/Entwicklung/Betatest) gelegt. Der geänderte Code liegt hier (https://github.com/following5/oc-server3/tree/new-listing-header).

Was mindestens noch zu tun ist:

- Alle Buttons auf gleiche Höhe und gleiche Schriftgröße bringen.
- Im Firefox wieder eine Leerzeile unter den Buttons einfügen.
- Das Layout im Internet Explorer reparieren:

Titel: Re: Shortlinks
Beitrag von: flopp am 14. April 2013, 20:52:41
Ich hab's noch ein wenig aufgebohrt, übersetzt, den Kartenlink korrigiert und auf den Testerver (http://wiki.opencaching.de/index.php/Entwicklung/Betatest) gelegt. Der geänderte Code liegt hier (https://github.com/following5/oc-server3/tree/new-listing-header).

Was mindestens noch zu tun ist:

- Alle Buttons auf gleiche Höhe und gleiche Schriftgröße bringen.
- Im Firefox wieder eine Leerzeile unter den Buttons einfügen.
- Das Layout im Internet Explorer reparieren:

ok, danke. Krass, dass der IE das so völlig anders darstellt. Aber wahrscheinlich ist mein Code auch nicht sooooo sauber...
Titel: Re: Shortlinks
Beitrag von: following am 26. April 2013, 00:32:48
Ich hab's jetzt erst mal wieder vom Testserver runtergenommen, weil ich sonst bei jedem Update Merge-Konflikte nachbearbeiten muss (wegen der Übersetzungen). Wenn die Punkte oben verbessert sind, spiele ich's wieder drauf.
Titel: Re: Shortlinks
Beitrag von: flopp am 26. April 2013, 07:24:17
OK. Ich habe gestern sogar ein bisschen dran gearbeitet.  Das Problem ist, dass sich der IE unglaublich zickig anstellt, während alle anderen Browser brav das machen, was ich möchte...
Titel: Re: Shortlinks
Beitrag von: flopp am 29. April 2013, 21:34:24
Git-Merge-Hölle!

Eigentlich ist es ziemlich peinlich, dass ich Git immer noch nicht auf die Reihe kriege...
Titel: Re: Shortlinks
Beitrag von: following am 07. Mai 2013, 16:17:21
Wir machen wir denn hier weiter?

Den Shortlink und den neuen Kartenlink könnte ich bereits übernehmen und freigeben, aber dann könntest du wieder einen Merge-Konflikt bekommen weil ich an den gleichen Codezeilen rumbastele. (Was eigentlich nicht schlimm ist, der Konflikt ist ja mit wenigen Handgriffen auflösbar)
Titel: Re: Shortlinks
Beitrag von: flopp am 07. Mai 2013, 16:41:14
Mach das mal. Falls mir das Mergen zu viel wird, schmeiße ich den Branch einfach weg,...
Titel: Re: Shortlinks
Beitrag von: flopp am 07. Mai 2013, 18:56:58
Wir machen wir denn hier weiter?

Den Shortlink und den neuen Kartenlink könnte ich bereits übernehmen und freigeben, aber dann könntest du wieder einen Merge-Konflikt bekommen weil ich an den gleichen Codezeilen rumbastele. (Was eigentlich nicht schlimm ist, der Konflikt ist ja mit wenigen Handgriffen auflösbar)

Ich werde mich heute abend noch mal dran setzen. Wann soll denn die nächste Version veröffentlicht werden?
Titel: Re: Shortlinks
Beitrag von: following am 07. Mai 2013, 19:24:35
Oh, jetzt habe ich schon damit begonnen. Hab das meiste von deinen Änderungen übernommen, noch einen Workaround für Ocprop eingebaut und die domainabhängige Konfiguration für die Shortlinks.

Die nächste Version hab ich erst in ein paar Wochen geplant, aber man könnte es auch vorab freigebn.
Titel: Re: Shortlinks
Beitrag von: following am 07. Mai 2013, 20:47:13
Also mit dem div-Layout bin ich überfordert ... aber ich hätte eine Tabellenvariante anzubieten:

https://github.com/following5/oc-server3/tree/new-listing-header2

Das ist jetzt neu von master abgeleitet; mit Shortlink auch in der Druckansicht. Buttons noch eckig ...
Titel: Re: Shortlinks
Beitrag von: flopp am 07. Mai 2013, 20:52:14
Also mit dem div-Layout bin ich überfordert ... aber ich hätte eine Tabellenvariante anzubieten:

https://github.com/following5/oc-server3/tree/new-listing-header2

Das ist jetzt neu von master abgeleitet; mit Shortlink auch in der Druckansicht. Buttons noch eckig ...

Der IE ist wieder mal maximal zickig!
Falls es mit einer Tabelle geht - warum nicht. Sieht doch so ganz gut aus...
Titel: Re: Shortlinks
Beitrag von: following am 07. Mai 2013, 21:38:55
Ok, ich hab das mal so in den Master übernommen und auf den Testerver gelegt. Falls du noch was verbessern willst, kannst du das ja von da aus machen.

Das sollte jetzt nochmal mit möglichst vielen Browser/Betriebssystem-Kombinationen getestet werden - vor allem auch mit mobilen Browsern, die zicken machmal auch.
Titel: Re: Shortlinks
Beitrag von: flopp am 07. Mai 2013, 21:43:17
Ok, ich hab das mal so in den Master übernommen und auf den Testerver gelegt. Falls du noch was verbessern willst, kannst du das ja von da aus machen.

Das sollte jetzt nochmal mit möglichst vielen Browser/Betriebssystem-Kombinationen getestet werden - vor allem auch mit mobilen Browsern, die zicken machmal auch.

Ok.

resource2/ocstyle/images/viewcache/14x14-gps-device.png existiert übrigens nicht...
Titel: Re: Shortlinks
Beitrag von: following am 07. Mai 2013, 21:46:46
Oops. Das kommt davon, wenn man git commit -am "..." macht, da werden keine neuen Dateien hinzugefügt.
Titel: Re: Shortlinks
Beitrag von: flopp am 07. Mai 2013, 21:49:42
Unter Linux sieht es mit Chrome & Firefox gut aus - auch mit unverschämt großer Nur-Text-Skalierung. Mit Handy & Tablet klappt auch alles.