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

Image provider #497

Closed
wants to merge 8 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@hjbflyer
Contributor

hjbflyer commented Apr 26, 2016

Hallo Andreas,
ich habe für einige Tabellenspalten Image-Provider (rote/grüne Pfeile) eingefügt. Man kann diese
Images auch noch für andere Spalten verwenden.

Alternativ könnte man auch Intervalle (3 fertig) definieren und die Pfeile rot, grün und schwarz(seitwärts)
machen.

@buchen

This comment has been minimized.

Owner

buchen commented Apr 26, 2016

Hi, Danke für die Change. Ich schaue sie mir die Tage näher an.

Wo hast Du die Icons her? Auf meinem Retina Mac sehen die etwas pixelig aus - vielleicht gibt es da noch eine grössere Auflösung? Ich habe die Icons bisher von Iconmonstr: http://iconmonstr.com

@hjbflyer

This comment has been minimized.

Contributor

hjbflyer commented Apr 27, 2016

Die Icons habe ich als Vektorgrafik erzeugt und dann exportiert. Vielleicht kann ich noch andere Export-Optionen versuchen. Als Anhang habe ich einen Screenshot mit meinen und einigen existierenden Icons
von meinem Retina iPad angehängt.

Frage wird auf dem Retina-Display das @2 Icon verwendet? Momentan habe ich die Icons auf einer 512x512 Bitmap gezeichnet. Ich kann das auch noch mal auf einer 1024x1024 Bitmap machen und
mit verschiedenen Pixelerzeugungs Algorithmen versuchen. Problematisch sind aber immer 45° Linien.

img_0042

@buchen

This comment has been minimized.

Owner

buchen commented Apr 27, 2016

Momentan habe ich die Icons auf einer 512x512 Bitmap gezeichnet. Ich kann das auch noch mal auf einer 1024x1024 Bitmap machen und mit verschiedenen Pixelerzeugungs Algorithmen versuchen.

Auch 512x512 müsste auf 32x32 ausreichend gut exportieren. Ich hatte gestern nicht genug Zeit um reinzuschauen... die @2 sollten auf allen HiDPI Screen genutzt werden. Das tut auch unter Windows.

@hjbflyer

This comment has been minimized.

Contributor

hjbflyer commented Apr 27, 2016

Noch eine Frage zur Klarstellung. Wie ich gesehen habe sind alle Icons 16x16 und die @2 jeweils 32x32. Ist das richtig?

@buchen

This comment has been minimized.

Owner

buchen commented Apr 27, 2016

So sollte es sein. Jetzt fällt mir auf: der Name stimmt nicht. Der Suffix muss "@2x" sein. Ansonsten werden die nicht gefunden.

@hjbflyer

This comment has been minimized.

Contributor

hjbflyer commented Apr 27, 2016

Das ist also die Ursache. Ich benenne die Daten um.

@buchen

This comment has been minimized.

Owner

buchen commented Apr 30, 2016

Merged.

Ich habe die Commits noch zu einem zusammengefasst - gesquashed wie es so schön bei Git heißt - und per Rebase auf meinen Master gesetzt.

@buchen buchen closed this Apr 30, 2016

@buchen

This comment has been minimized.

Owner

buchen commented Apr 30, 2016

BTW, Checkstyle hat ein paar unbenutzte imports gemeldet: 8f70718

In Eclipse kann man sogenannte "Save Actions" aktivieren (leider nur per Workspace, nicht per Projekt) mit denen die Imports automatisch bei Speichern aktualisiert werden. Dabei wird der Code Formatter verwendet, der im Projekt konfiguriert ist.

bildschirmfoto 2016-04-30 um 07 19 01

@hjbflyer

This comment has been minimized.

Contributor

hjbflyer commented Apr 30, 2016

Ich mag diesen Automatismus nicht so gern. Leider habe ich die unused Imports übersehen, weil
mir die Warning-Flags nicht aufgefallen sind, da es ja einige "Discouraged access" gibt.

Wie schon mal geschrieben, ist für mich das Setzen der '{' auf eine neue Zeile sehr unüblich (erinnert
mich mehr an C und C++). Ich schätze die Mehrheit folgt dem SUN-Code-Style. Wahrscheinlich folgt
als Sicherheit aus diesem Style, dass selbst if-then-else, etc. mit nur einem Statement immer geklammert werden.

@buchen

This comment has been minimized.

Owner

buchen commented May 1, 2016

"Discouraged access"

Die sind der Tat nervig. Ich habe die "Issues" view nach Issue Type gruppiert. Dann sehe ich sofort ob es neben diesen discouraged access Hinweisen noch weitere gibt.

Eclipse e4 hat sich noch nicht entschieden, dass diese Methoden das finale API sind. Es gibt aber auch keine Alternativen. Bisher habe ich diese Meldungen nicht unterdrückt weil ich hoffte bald auf das finale API umziehen zu können. Das ist aber auch schon ein paar Jahre her...

buchen added a commit that referenced this pull request May 1, 2016

@buchen

This comment has been minimized.

Owner

buchen commented May 1, 2016

@hjbflyer Ich habe mir erlaubt die Icons leicht anzupassen - siehe d582bc4 Die Idee mit den Icons finde ich super. Visuell waren die mir etwas zu "schwergewichtig". Am Ende ist das Geschmacksache, das weiß ich. 😄

@hjbflyer

This comment has been minimized.

Contributor

hjbflyer commented May 2, 2016

Mir gefallen Deine Icons auch besser.

@hjbflyer hjbflyer deleted the hjbflyer:image-provider branch Aug 17, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment