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

Bestell Info in Zwischenablage kopieren #43

Merged
merged 5 commits into from
May 31, 2016
Merged

Bestell Info in Zwischenablage kopieren #43

merged 5 commits into from
May 31, 2016

Conversation

zuphilip
Copy link
Member

Dies ist eine erste Version zu #35. Es müsste bereits funktionieren, aber gibt noch etliches zu tun:

  • Gerne würde ich die Anzeige mit einem Tooltip oder ähnlichen machen. Das habe ich aber auf Anhieb nicht verstanden wie dies geht...
  • Ebenfalls würde ich gerne ein Symbol neben dem Text hinzufügen etwa Bestell Info hinzufügen (oder ein ähnliches). Müsste man aber noch prüfen ob frei ist...
  • Die COinS Daten habe ich einmal in die Hauptseite kopiert. Wahrscheinlich funktioniert dies aber nur bedingt, da ja dann die Daten erst beim Nachladen erscheinen und damit meist für die entsprechenden Literaturverwaltungsprogramme nicht abgreifbar sind...
  • mehr Testen
  • Code aufräumen

@zuphilip
Copy link
Member Author

zuphilip commented May 23, 2016

Symbol stammt aus https://github.com/github/octicons d.h. Font unter SIL (damit wohl kein Problem zum Wiederverwenden). Nennung nicht vergessen...

@zuphilip
Copy link
Member Author

Okay, so sieht es momentan aus:
clipboard

Leider ist der Wechsel zu Kopiert! nach dem Klick noch etwas Ad-Hoc und nicht unbedingt gut sichtbar. Hat jemand eine Idee wie man dies besser machen könnte?

@kba
Copy link
Contributor

kba commented May 23, 2016

I tried the clipboard.js branch, merged it with master, built the docker container but when I run it, I see no effect.

But it does work, the data is copied to the clipboard.

Also, there's an error GET http://localhost:12345/malibu/isbn/isbn2MANholdings.php?isbn=978-3894015374 404 (Not Found)

@zuphilip
Copy link
Member Author

but when I run it, I see no effect.

What do you mean? It looks like the picture above with the tooltip, right? Moreover, if you copy it then the tooltip text is changed, but this is hardly visible...

@kba
Copy link
Contributor

kba commented May 24, 2016

Ah, I see, the tooltip text is changed, yes. But that is a bit too subtle for me :-)

Maybe use a fixed notification with setTimeout? Here's a quick try at this, looks like this:

screenshot - 05242016 - 09 16 38 am

@zuphilip
Copy link
Member Author

Yeah, that is a good idea. I will continue on your commit here, maybe also some time offline now.

The same mechanism could also be used in #47 maybe with a red background.

@zuphilip
Copy link
Member Author

Okay, I think this should be ready now. More tests and feedback are welcome. CC @kba @tkees

@zuphilip zuphilip merged commit eae179a into master May 31, 2016
@zuphilip zuphilip deleted the clipboard.js branch May 31, 2016 20:44
@zuphilip
Copy link
Member Author

@kba Thank you for the help with this!

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.

2 participants