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

Line145 . ---> .secmenu span :not(.newsin) #284

Merged
merged 1 commit into from Aug 30, 2017

Conversation

Projects
None yet
4 participants
@bbfriend
Contributor

bbfriend commented Aug 27, 2017

Text color (span tag) is not displayed correctly in News BOX.

.secmenu span ---> .secmenu span :not(.newsin)

navi_css line145

Line145 . ---> .secmenu span :not(.newsin)
.secmenu span ---> .secmenu span :not(.newsin)
@frase-git

This comment has been minimized.

Show comment
Hide comment
@frase-git

frase-git Aug 28, 2017

+1

That was my fault.

frase-git commented Aug 28, 2017

+1

That was my fault.

@cmb69 cmb69 merged commit b87b176 into cmsimple-xh:master Aug 30, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@cmb69

This comment has been minimized.

Show comment
Hide comment
@cmb69

cmb69 Aug 30, 2017

Member

Thanks, Takashi!

Member

cmb69 commented Aug 30, 2017

Thanks, Takashi!

@frase-git

This comment has been minimized.

Show comment
Hide comment
@frase-git

frase-git Oct 28, 2017

Sorry, da habe ich damals nicht richtig aufgepasst.
Die Korrektur im Stylesheet ist nötig, aber so falsch.
Die aktiven Menüpunkte verlieren die Hervorhebung:
fhs-simple-navi-css
Links der Ist-Zustand. Rechts Das Soll.

Die Korrektur muss also lauten, in navi.css Zeile 145:
.secmenu span:not(???) {

STOPP, bitte noch ein paar Minuten warten ...
Das wird wohl ein größerer Umbau, da :not nur einen einzigen Negationsselektor enthalten darf.

frase-git commented Oct 28, 2017

Sorry, da habe ich damals nicht richtig aufgepasst.
Die Korrektur im Stylesheet ist nötig, aber so falsch.
Die aktiven Menüpunkte verlieren die Hervorhebung:
fhs-simple-navi-css
Links der Ist-Zustand. Rechts Das Soll.

Die Korrektur muss also lauten, in navi.css Zeile 145:
.secmenu span:not(???) {

STOPP, bitte noch ein paar Minuten warten ...
Das wird wohl ein größerer Umbau, da :not nur einen einzigen Negationsselektor enthalten darf.

@frase-git

This comment has been minimized.

Show comment
Hide comment
@frase-git

frase-git Oct 28, 2017

.secmenu span {
	padding: .5em .5em .5em 1em;
	display: block;
	background: rgba(0, 0, 0, 0.05);
	color: #f60;
}
.secmenu .news span {
	padding: 0;
	display: inline;
	background: transparent;
}

Das wäre erstmal eine vorübergehende Lösung - aber sehr schlecht :(
Für span im Text geht es, aber es kann noch alle möglichen Varianten von span geben. Was passiert dann?
Ideen?

frase-git commented Oct 28, 2017

.secmenu span {
	padding: .5em .5em .5em 1em;
	display: block;
	background: rgba(0, 0, 0, 0.05);
	color: #f60;
}
.secmenu .news span {
	padding: 0;
	display: inline;
	background: transparent;
}

Das wäre erstmal eine vorübergehende Lösung - aber sehr schlecht :(
Für span im Text geht es, aber es kann noch alle möglichen Varianten von span geben. Was passiert dann?
Ideen?

@cmb69

This comment has been minimized.

Show comment
Hide comment
@cmb69

cmb69 Oct 28, 2017

Member

Hm, warum sind denn die Newsboxen überhaupt Teil von .secmenü? Hat das einen tieferen Sinn?

Und wenn es nur darum geht, die <span>s im eigentlichen Menü hervorzuheben, warum nicht

.secmenu .sdoc span, .secmenu .sdocs span {…}
Member

cmb69 commented Oct 28, 2017

Hm, warum sind denn die Newsboxen überhaupt Teil von .secmenü? Hat das einen tieferen Sinn?

Und wenn es nur darum geht, die <span>s im eigentlichen Menü hervorzuheben, warum nicht

.secmenu .sdoc span, .secmenu .sdocs span {…}
@frase-git

This comment has been minimized.

Show comment
Hide comment
@frase-git

frase-git Oct 28, 2017

Hm, warum sind denn die Newsboxen überhaupt Teil von .secmenü? Hat das einen tieferen Sinn?

Mit Sicherheit. Bei mir hat alles einen Sinn. Nur, fällt der mir gerade nicht ein ;-)

Also, dein Vorschlag ist perfekt. Danke.
Zeile 145:
.secmenu .sdoc span, .secmenu .sdocs span {

Ich hatte mit menulevel* rumprobiert und bin auf keinen grünen Zweig gekommen. Wenn solche Probleme auftauchen, sehe ich manchmal den Wald vor lauter Bäumen nicht :(

frase-git commented Oct 28, 2017

Hm, warum sind denn die Newsboxen überhaupt Teil von .secmenü? Hat das einen tieferen Sinn?

Mit Sicherheit. Bei mir hat alles einen Sinn. Nur, fällt der mir gerade nicht ein ;-)

Also, dein Vorschlag ist perfekt. Danke.
Zeile 145:
.secmenu .sdoc span, .secmenu .sdocs span {

Ich hatte mit menulevel* rumprobiert und bin auf keinen grünen Zweig gekommen. Wenn solche Probleme auftauchen, sehe ich manchmal den Wald vor lauter Bäumen nicht :(

@lck-git

This comment has been minimized.

Show comment
Hide comment
@lck-git

lck-git Oct 28, 2017

Oder einfach
.secmenu ul span {..}

oder evtl.
.secmenu > ul span

lck-git commented Oct 28, 2017

Oder einfach
.secmenu ul span {..}

oder evtl.
.secmenu > ul span

@frase-git

This comment has been minimized.

Show comment
Hide comment
@frase-git

frase-git Oct 28, 2017

Oder einfach
.secmenu ul span {..}

Was ist, wenn in der newsbox eine ul vorkommt ?
sdoc und sdocs sind eindeutiger - oder?

frase-git commented Oct 28, 2017

Oder einfach
.secmenu ul span {..}

Was ist, wenn in der newsbox eine ul vorkommt ?
sdoc und sdocs sind eindeutiger - oder?

@lck-git

This comment has been minimized.

Show comment
Hide comment
@lck-git

lck-git Oct 28, 2017

So passiert nichts :)
.secmenu > ul span

lck-git commented Oct 28, 2017

So passiert nichts :)
.secmenu > ul span

@cmb69

This comment has been minimized.

Show comment
Hide comment
@cmb69

cmb69 Oct 28, 2017

Member

.secmenu > ul span

Ja, das geht bei der jetzigen Struktur auch gut. Ich selbst finde die .sdoc(s) Variante aber sprechender, weil es ja wirklich nur darum geht, die aktuell gewählte Seite im Menü hervorzuheben. Aber ich finde, Frank sollte das entscheiden – ist schließlich sein Template. :)

Member

cmb69 commented Oct 28, 2017

.secmenu > ul span

Ja, das geht bei der jetzigen Struktur auch gut. Ich selbst finde die .sdoc(s) Variante aber sprechender, weil es ja wirklich nur darum geht, die aktuell gewählte Seite im Menü hervorzuheben. Aber ich finde, Frank sollte das entscheiden – ist schließlich sein Template. :)

@frase-git

This comment has been minimized.

Show comment
Hide comment
@frase-git

frase-git Oct 28, 2017

Huch?! Entscheiden ...
.sdoc(s)

frase-git commented Oct 28, 2017

Huch?! Entscheiden ...
.sdoc(s)

cmb69 added a commit that referenced this pull request Oct 28, 2017

@cmb69

This comment has been minimized.

Show comment
Hide comment
@cmb69

cmb69 Oct 28, 2017

Member

Erledigt mit a3012ed

Member

cmb69 commented Oct 28, 2017

Erledigt mit a3012ed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment