[bug] Blok met code valt in DM buiten beeld

[ThinkPad]

Ik deel wel eens wat handige domotica gerelateerde scriptjes met medetweakers en eigenlijk heb ik altijd moeite om het te lezen. De wat langere regels vallen namelijk buiten beeld. Nu is het volgens mij met Safari zo dat je een horizontale scrollbalk krijgt, maar op mijn Windows PC met IE11 / Chrome of Firefox is dat niet het geval.



Is hier misschien een oplossing voor mogelijk? Dat er een 'select all' knopje komt, of bijv. een horizontale slider?

[Hero of Time]

Dit komt eerder omdat de CSS voor de code block dat gebruikt wordt hetzelfde is voor normale posts. In DMs is het bericht minder breed, met dit dus tot gevolg. Er zijn 2 oplossingen: maak de breedte van de code blocken relatief (% breedt van parent) ipv absoluut (600px breed bijvoorbeeld) of specificeer voor DMs apart hoe code blocken worden weergegeven. Dat laatste wordt tenslotte al gedaan voor het bericht zelf.

Naast bovenstaande is natuurlijk ook nog de overflow. Die treedt pas op als de tekst langer is dan de breedte dat in de css staat. Blijkbaar fixt Safari dat zelf (waarschijnlijk omdat het kijkt naar de parent div), maar andere browsers doen dat niet.


Wil je nu gelijk een oplossing, dan stel ik voor dat je 't via custom CSS doet. Sterker nog, er is al zoiets in de code, het wordt alleen overschreven door een regel later.

Cascading Stylesheet:

.codewrap > table.phphighlight {   /* forum.css:465 */
width: 800px;
}

.codewrap table.phphighlight {     /* forum.css:464 */
width: 100%;
}

Zet die 800px dus op 100% (eventueel met !important erbij als 't niet werkt) en je bent er. Waarom er is besloten voor een specifieke breedte, geen idee.

[ThinkPad]

Thanks, custom CSS is voor nu wel een workaround.
Heb jouw stukje gekopieerd (custom CSS feature gebruik ik al) en de 800px veranderd in 100%. Maar het werkt niet (met of zonder '!important').

Cascading Stylesheet:

/* Fix weergave van codeblocks in DM thread */
.codewrap > table.phphighlight {   /* forum.css:465 */
width: 100% !important;
}

.codewrap table.phphighlight {     /* forum.css:464 */
width: 100% !important;
}

[Hero of Time]

Ik heb de code overgenomen van wat ik kon vinden in de CSS van tweakers zelf. Je hebt alleen het stukje nodig waar 800px stond. Net zelf met mijn eigen CSS getest en ik krijg het gewenste resultaat, iig hier in normale posts, met het volgende:

Cascading Stylesheet:

.codewrap > table.phphighlight { 
    width: 80% !important;
}

Misschien dat je nog met Firebird of whatever moet achterhalen wat de class is van de div.

[ThinkPad]

Er wijzigt niks

Ik voel mij echt te dom hiervoor

[Hero of Time]

Ik heb je net een DM gestuurd, want ik kan 't niet zo snel zien in mijn eigen DMs.

[NMe]

Ik vraag me af hoeveel zin het heeft om veel moeite te gaan steken in een custom CSS-oplossing terwijl dit eigenlijk gewoon een oversight is die de devvers waarschijnlijk zullen willen fixen?

[Hero of Time]

Ik wil weten wat er in de CSS fout gaat om op die manier de devs te helpen te achterhalen waar 't mis gaat. Ik kan het namelijk niet reproduceren.

[Glodenox]

Zou iemand me in DM een gelijkaardig stuk code kunnen opsturen? Dan kan ik er ook eens naar kijken.

[Hero of Time]

Ik heb het net via DM met Thinkpad getest op Linux met Opera 12.16 en Vivaldi 1.0.190.2, en op Windows 8.1 x86 met IE 11 (met de laatste updates, ook die van vandaag) en geen enkel probleem gezien, gewoon netjes scrollbalken waar 't moet.

[crisp]

Wellicht maakt de instelling voor 'Uitklap codeblokken uitschakelen' in je voorkeuren nog uit?

[Hero of Time]

Dat moet 't wel zijn, want ik heb 'm uit staan, maar zodra ik 't vinkje aanzet, loopt 't ook van 't scherm af.

[crisp]

Misschien moeten we die instelling gewoon eens weggooien en het voor iedereen hetzelfde doen

[ThinkPad]

Door die instelling uit te zetten is het gefixed

Staat die default ingeschakeld? Misschien idd beter om hem uit te zetten, uitgeschakeld (en dus horizontale scrollbar) is een stuk makkelijker.

[crisp]

Volgens mij staat hij default uit

[NMe]

Stiekem is die setting eigenlijk bloat, dus die zou ik ook gewoon schrappen.

[_David_]

NMe schreef op woensdag 10 juni 2015 @ 11:46:
Stiekem is die setting eigenlijk bloat, dus die zou ik ook gewoon schrappen.

Dat kan je over meerdere settings zeggen