search

Hamcha / maud

rocks
https://crunchy.rocks/
Other
5 stars 1 forks source link

Site options hiding + quote button behaviour change + minifix shortify #87

Closed silverweed closed 8 years ago

silverweed commented 8 years ago

Ti faccio revieware i commit perché sono stanco morto e potrei aver scritto cagate...vedi #86

Edit: ho aggiunto un commit che sistema un problema di shortify che si presenta attualmente ad esempio qui: se il post break è fatto a metà di una tag, questa non viene chiusa correttamente. Il nuovo comportamento è droppare tutta la tag in questione e chiudere quelle rimaste sullo stack.

P.S. se vedi che gli import cambiano ordine è perché ho messo il plugin goimports al posto di gofmt, e ha uno stile diverso che mette separati tutti gli import non standard.

Hamcha commented 8 years ago

Non so cosa volevi sistemare con il quote button, ma il nuovo behavior taglia il testo se si usa selezione a blocco, ho fatto delle modifiche (4b82337) in modo che non lo faccia e che metta a capo (altrimenti non conta come quote). Se gli dai un'occhiata magari evitiamo di mergare mie eventuali cazzate (specie non sapendo il motivo del fix originale).

Il resto LGTM :+1:

silverweed commented 8 years ago

Volevo fare in modo che quotasse un eventuale testo selezionato da un qualunque post invece di mettere semplicemente un '>': il comportamento attuale è:

Quello desiderato sarebbe:

Hamcha commented 8 years ago

Ho testato la modifica localmente e mi pare funzionare. Come gia detto prima per me è un OK. Visto che non ho notato obiezioni nella tua risposta (ed è passato un bel po') direi che lo mergo e nel caso vediamo di fixare eventuali regression..