LSS-Manager / lss-manager-v2

Diese Version wird nicht mehr Supported!
www.lss-manager.de
2 stars 0 forks source link

Code Qualität #12

Closed eaglefsd closed 8 years ago

eaglefsd commented 8 years ago

BITTE geht euren Code mit JSLint durch. Von mir aus kann ich auch nachher einen Pull Request mit den Verbesserungen machen, aber im Moment habt ihr zahlreiche Fehler im Code. Ja, Fehler, nicht nur designtechnisch.

eaglefsd commented 8 years ago
var org_building_maps_draw = building_maps_draw;
building_maps_draw = function (e) {
    //org_building_maps_draw(e);
    var t = L.marker([e.latitude, e.longitude], {
        title: e.name,
        icon: icon_empty,
        riseOnHover: true
    }).bindLabel(e.name).addTo(map);
    t.building_id = e.id, "undefined" != typeof e.opacity && t.setOpacity(e.opacity), iconMapGenerate(e.building_marker_image, t), t.on("click", function () {
        lightboxOpen("/buildings/" + e.id)
    }), building_markers.push(t)
    $.each(building_markers, function (key, value) {
        value.hideLabel();
        value.unbindLabel();
        cars = car_list(value.building_id);
        data = value.options.title;
        if (cars.length > 0) {
            data += '&nbsp;<i class="fa fa-car"></i>' + cars.length;
            data += car_list_printable(cars);
        }
        value.bindLabel(data, {zIndex: 999});
    });
}

Hier geb ich auf. Keine Ahnung, was das soll, aber das kann so nicht stimmen.

eaglefsd commented 8 years ago

https://github.com/lostdesign/lss-layout-manager/pull/13