Skip to content
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

altezza diversa dei MEPbox #30

Open
lucacorsato opened this issue Sep 26, 2012 · 8 comments
Open

altezza diversa dei MEPbox #30

lucacorsato opened this issue Sep 26, 2012 · 8 comments
Labels

Comments

@lucacorsato
Copy link

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
Copy link
Member

@dottorblaster Puoi buttargli un occhio ?

@dottorblaster
Copy link
Member

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
Copy link
Author

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
Copy link
Member

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

@claudiodangelis
Copy link
Contributor

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
Copy link
Member

Dato che abbiamo risolto, chiudo il bug. :)

@claudiodangelis
Copy link
Contributor

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

@paolomainardi
Copy link
Member

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

@paolomainardi paolomainardi reopened this Oct 3, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants