Skip to content
This repository has been archived by the owner on Jan 1, 2020. It is now read-only.

Metainfo css fix #84

Merged
merged 1 commit into from Jan 29, 2013
Merged

Metainfo css fix #84

merged 1 commit into from Jan 29, 2013

Conversation

ghost
Copy link

@ghost ghost commented Jan 28, 2013

Prob hier:
css_issues

@ghost
Copy link
Author

ghost commented Jan 28, 2013

yup, genau. überlagert z.b. das hauptmenü. tritt nur auf wenn man ein metainfo vergeben hat bzw. der button dafür da ist.

@staabm
Copy link
Member

staabm commented Jan 28, 2013

Könntest du bitte nen screenshot hochladen wo man den aktuellen Fehler sieht und wie es danach aussieht?

Welcher Browser?

@ghost
Copy link
Author

ghost commented Jan 28, 2013

naja, der erste screen oben zeigt doch schon das prob.

hier gefixt:
metainfo_cssfix

ich nutze die neuste version von firefox unter ubuntu

@jdlx
Copy link
Contributor

jdlx commented Jan 28, 2013

@staabm der bug is (alt)bekannt und confirmed. Evtl. noch @tbaddade fragen ob er Gründe für ein Veto sieht, und wenn nicht -> rein damit.

@ghost
Copy link
Author

ghost commented Jan 28, 2013

scheint definitiv ein ff prob zu sein. bei ff 3.6 unter windows tritts auch auf. bei ie, safari allerdings net...

@ghost
Copy link
Author

ghost commented Jan 28, 2013

übrigens taucht bei mir unter ubuntu ein weisser balken wie im screenshot zu sehen auf und die weisse linie direkt unter "Hauptmenü" und "Addons" ist doppelt so dick. wenn ich die standard-schriftgröße unter firefox von 17 auf 16 (geht hier wegen font-size: 100%) umstelle ist es zwar ok aber dann ist mir die schrift zu klein. deshalb habe ich einfach padding von footer und font-size von den überschriften geändert:

div#rex-navi-main dt {
    font-size: 16px;
}

div#rex-footer {
    padding: 8px 0;
}

@jdlx
Copy link
Contributor

jdlx commented Jan 28, 2013

Ich würde mit irgendwelche partikular-fixes zurückhaltend sein.. der outline bug ist klar/eindeutig, aber jetzt generelle CSS tweak Fäßer würd ich eher zulassen.. allein weil das notwendige Geteste across Browser so aufwändig käme.

@ghost
Copy link
Author

ghost commented Jan 28, 2013

@jdlx passt. deshalb hab ichs auch net in den pr gepackt. aber erwähnen wollt ichs trotzdem mal...

@staabm
Copy link
Member

staabm commented Jan 28, 2013

Ich seh den Fehler im Screenshot nicht.

@jdlx
Copy link
Contributor

jdlx commented Jan 28, 2013

Mußt den screenshot in groß ankucken.. die komplette outline um den Kategorienblock ziehts nach links über navi hinaus auf..

@ghost
Copy link
Author

ghost commented Jan 28, 2013

beim nächsten mal mach ich nen bunten pfeil mit dazu ;)

@jdlx
Copy link
Contributor

jdlx commented Jan 28, 2013

-> AwesomeScreenshot ..

@ghost
Copy link
Author

ghost commented Jan 28, 2013

@jdlx guter tip, hatt ich ganz vergessen...

@gharlan
Copy link
Member

gharlan commented Jan 29, 2013

@tbaddade mergen?

tbaddade added a commit that referenced this pull request Jan 29, 2013
@tbaddade tbaddade merged commit ed722ac into redaxo:master Jan 29, 2013
@ghost ghost deleted the metainfo_cssfix branch January 29, 2013 14:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants