spaghetti-open-data / tweetyourmep

TweetyourMEP
GNU Affero General Public License v3.0
17 stars 11 forks source link

altezza diversa dei MEPbox #30

Open lucacorsato opened 12 years ago

lucacorsato commented 12 years ago

nella visualizzazione verticale da tablet (iPad) l'altezza dei MEPbox non è uguale per tutti. il testo all'interno non si ridimensiona in base alla larghezza disponibile

paolomainardi commented 12 years ago

@dottorblaster Puoi buttargli un occhio ?

dottorblaster commented 12 years ago

Purtroppo sulla visualizzazione iPad non si può fare niente, dipende da Bootstrap ma non scala correttamente. Per ora marco come wontfix anche se continuo a tenere d'occhio la cosa. (tengo aperta la issue)

lucacorsato commented 12 years ago

controllato anche su galaxy 10.1... stessa cosa... non ho molta familiarità con bootstrap per essevi d'aiuto, ma non è forse solo un problema di scala dei font? sempre che si voglia un font scalabile...

paolomainardi commented 12 years ago

@lucacorsato Stiamo usando uno di questi temi bootstrap, potresti fare una verifica qui per favore ? http://bootswatch.com/

claudiodangelis commented 12 years ago

Questa soluzione prende l'altezza del box più 'alto' e normalizza a posteriori tutte le anteprime (richiede jQuery):

Array.max = function( array ){return Math.max.apply( Math, array );};

var anteprime = document.getElementsByClassName("thumbnail");
var arr = new Array();
for (var i = 0; i < anteprime.length; i++) {
    arr.push(anteprime[i].offsetHeight);
};

$(".thumbnail").css("height",Array.max(arr));
dottorblaster commented 12 years ago

Dato che abbiamo risolto, chiudo il bug. :)

claudiodangelis commented 12 years ago

Wait @dottorblaster! prima di chiudere cerchiamo una soluzione migliore, con questa si può incorrere in disastri come in questo screenshot

paolomainardi commented 12 years ago

Il fix in effetti non è nel master, discutiamone ancora la issue #30, per adesso abbiamo pushato solo la #42