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
"Open in Browser" konfigurierbar machen #309
Comments
👍 Sehr gute Idee. Ich bin selber nie dazu gekommen, weil... na ja... ich kann sie ja einfach hinzufügen... Alle Punkte (eins bis drei) sehe ich auch so. Die einzige Frage für mich: Wo speichern? Als "preference" im "workspace" Verzeichnis? Das gilt dann für alle geöffneten Dateien, aber eben nur in dieser Installation. Oder im "client"? Das ist dann pro Datei, die Datei bei Änderungen als "dirty" markiert, aber wenn man die Datei auf einem anderen Rechner öffnet, sind die Kontenmenüs da. Was meinst Du? Dann fände es super, wenn man auch einen Separator hinzufügen könnte. Sagen wir mal in dem der Bookmark Name einfach ein "-" dash ist:
Momentan werden nur isin und tickerSymbol unterstützt. Ich würde mindestens noch wkn hinzufügen, vielleicht noch name (als Suchanfrage). |
Ich hätte die WebLocations im inifile gespeichert, gleich wie die Spracheinstellung. Aber irgendwie wenn ich so an speichern in der Cloud denke ist die Idee die Bookmarks im Client zu speichern bestechend!
|
Guter Punk. Vielleicht könnte man in der Navigationszeile unter "Allgemeine Daten" noch einen Punkt "Einstellungen" einfügen. Da sind dann momentan nur die Bookmarks. Falls mehr kommt, könnte man ja noch mehr Reiter erstellen.
Im Client gibt es schon ein Attribut "properties" für generische key-value pairs - allerdings als Strings. Entweder Du speicherst Die Bookmarks da - in irgendeiner Form - serialisiert. Oder man bohrt das auf und macht eine Map<String, Object>. Ich würde ersteres mal ausprobieren. Wenn das zu umständlich wird, dann eher aufbohren. Mein Hintergedanke: ich möchte eine Proliferation von Attributen im Client vermeiden.
Die Spracheinstellung ist dann spezifisch weil die Local vor dem Start gesetzt werden muss. Darum ist die in dem "inifile". Andere Einstellungen kann man auch separat speichern (die landen dann im Workspace Folder). |
Ah ja, und die Change kann ruhig gegen master gehen. Dafür brauchen wir nicht auf den feature_currencies branch warten. |
Hi Andreas, klar geht der gegen Master. Ich habe nun einen ersten Entwurf. Es fehlt Es fehlt mir im TableViewer noch eine möglichkeit die WebLocations |
So einen Event gibt es nicht. Bisher habe ich bei solchen Fällen bei der Änderung einfach "drüber" serialisiert. In dem Fall der "Bookmarks" ändert sich das ja auch nicht häufig - nur wenn der Benutzer in der Einstellungenansicht tatsächlich eine Änderung vornimmt. Kann man da nicht z.B. die ClientSettings#toString in die Client Properties schreiben? Zu simplistisch? Ansonsten müsste man einen Event einführen. Und eine - wie auch immer geartete - Registrierung von Listeners.
Genau. Lieber kleine Inkremente. |
Ich sehe gerade Du hast schon was in Dein Repo gepusht. Mir ist noch die Idee gekommen, dass man die Konfiguration ja auch aus dem Kontextmenü aufrufen könnte. Also ein letzter Menüpunkt "Bookmarks konfigurieren..." oder so ähnlich. Dann ist die Konfiguration auch an der Stelle, an der man sich gerade denkt, Mensch, ich würde doch noch andere Bookmarks haben wollen. |
gute Idee ! Das kann ich gerne noch hinzufügen. Hast du eine Idee wie man vom WebLocationMenu an eine Referenz auf das PortfolioPart kommt ? Denn da müsste ich wohl activateView(view, parameter) aufrufen. |
Die |
- bumped up file version because older versions will not be able to properly load and save a file with client settings - added context menu to add separator so that users get a hint how to create separators - added NON-NLS-x tags - used IMG_PLUS icon in toolbar to be consistent with other views - removed icons in context menu to be consistent with other views - fixed labels: twice "Nachher einfügen" Issue: #309
Im Prinzip fehlt jetzt eigentlich nur ein "drag & drop" um die Bookmarks in die richtige Reihenfolge zu bringen. |
Ich würde gerne zusätzliche Webseiten zum Kontext Menu hinzufügen. Dazu sollten die Webseiten konfigurierbar sein.
Idee:
The text was updated successfully, but these errors were encountered: