citmp2015 / flink-visual-programming

2 stars 9 forks source link

Frontend redesign #108

Closed alxkum closed 8 years ago

alxkum commented 8 years ago

Issue #94 and #101 closed. Screenshot.
I tried not to touch the actual template files but it was sometimes necessary. Also, the responsiveness is kind of broken in the original template and some other minor things (e.g. when the viewport is longer than the window height && all items from the sidebar are expanded && the sidebar is longer than the canvas => canvas will not correctly stretch to the bottom of the viewport because the sidebar is clipped/fixed to the header (lol) and therefore ignored for correct height calculations of other elements). I will need to create a new template for the layout because the old one has too many issues.
Please also take a closer look at the somewhat heavily modified Gruntfile.
To distinguish the source and sink components from within CSS, the app.factory.js had to be slightly modified (adds a special class to source and sink components when creating/rendering them).
Please also stop using .btn-warning for Cancel buttons in modal dialogs :)

MonsJovis commented 8 years ago

Wow! Auf dem Screenshot schauts ja wirklich cool aus! Ich schau mir dann die Tage noch den Code an. Könntest du inzwischen noch den merge conflict lösen?

alxkum commented 8 years ago

Öhm, ich weiß ehrlich gesagt nur wie man den conflict löst beim mergen. Dann würde ich den PR aber ja selber akzeptieren. So weit ich das sehe ist der Konflikt weil in der index.html unten der Pfad zu jQuery leicht abgeändert wurde, während bei mir noch der alte ist.
Edit: Oder soll ich versuchen einen neuen PR zu machen?

MonsJovis commented 8 years ago

probier mal den (aktuellen) dev branch in deinen branch zu mergen, danach den konflikt zu lösen und committen.

MonsJovis commented 8 years ago

noch was: da der build nicht durch ist, schätze ich, dass JSHint schreit. Lass mal grunt test im client-ordner drüberlaufen.

alxkum commented 8 years ago

Done and done. Der build war aber schon vor meinem commit gefailed, afaik.