Poprawka kosmetyczna - "Lubię to!"

[iVenoom]

Witam.

Ostatnio zauważyłem, że gdy klikniemy Lubię to! obrazek ma mały błąd.

Chodzi o prostą poprawkę graficzną .

[Vomar™]

He, myślałem że tylko ja tak mam ;o

 

Jasne, że tak.

[3nichilat0]

Wszystko przez :

.ipsLikeButton:before {
   content: "✓ ";
   font-weight: bold;
}

w ipb_styles.css

 

Co do sugestii to TAK.

[szelkoffy]

tak, brzydko to wygląda.

[Savien]

tak

[Gość]

Mam to samo więc na

 

TAK. 

[Ksawiryn]

Tak.

[Sℓαѕнєя_]

He, myślałem że tylko ja tak mam ;o

 

Jasne, że tak.

[Falem]

 

He, myślałem że tylko ja tak mam ;o

 

Jasne, że tak.

 

[Gość]

Wszystko przez :

 

.ipsLikeButton:before {
   content: "✓ ";
   font-weight: bold;
}

w ipb_styles.css

 

Co do sugestii to TAK.

Dzięki temu masz ten znaczek przy lubię to.

[3nichilat0]

Dzięki temu masz ten znaczek przy lubię to.

Tylko poco jak w kodzie macie na sztywno dodane bo niema zdefiniowanego 'Lubię to' i 'Nie lubię' w pliku językowym:

<li style="display:none">
	<a class="ipsLikeButton ipsLikeButton_enabled rep_up" href="adres" title="Lubię to!">
		Lubię to
	</a>
</li>
<li>
	<a class="ipsLikeButton ipsLikeButton_disabled rep_down" href="adres" title="Nie lubię tego :(">
		Nie lubię
	</a>
</li>

a JS zmienia tylko style dla <li> na 

style="display: none;"

dla nieaktywnego <li> + dodaje

 ::before

dla aktywnego <li>.

więc cez tego w CSS takie coś wystarczy :

<li style="display:none">
	<a class="ipsLikeButton ipsLikeButton_enabled rep_up" href="adres" title="Lubię to!">
		✓ Lubię to
	</a>
</li>

No chyba że łatwiej będzie wam przeanalizować [co jest lepsze na dłuższą metę] prototype.js czy tak ipb.js

 

Ps. Nie wiem jak wy to zrobiliście ze zmianą koloru 'lubię to' ale kod dalej pobiera te zielone tabelki ale ich nie wczytuje.

 

•   Dolna tabelka działa dla 'Nie lubię' górna nie jest wykorzystywana.

&

•  W ogóle nie jest wykorzystywana.

////////Edit

 

To nie lepiej zrobić 1 porządne rozwiązanie?

 

Zmienić:

        .ipsLikeButton.ipsLikeButton_enabled {
                background: #2983B8;
                /*border: 1px solid #5ea128;*/
        } 

Na:

        .ipsLikeButton.ipsLikeButton_enabled {
                background: #2983b8 url('http://www.mpcforum.pl/public/style_images/def342/like_button.png') no-repeat 0;
                /*border: 1px solid #5ea128;*/
        }

A obrazek:

• 

  http://www.mpcforum.pl/public/style_images/def342/like_button.png

Zmienić na np:

• 

• http://www.mpcforum.pl/uploads/images/734671413279240638160.png
  

[Gość]

Tylko poco jak w kodzie macie na sztywno dodane bo niema zdefiniowanego 'Lubię to' i 'Nie lubię' w pliku językowym:

 

<li style="display:none">
	<a class="ipsLikeButton ipsLikeButton_enabled rep_up" href="adres" title="Lubię to!">
		Lubię to
	</a>
</li>
<li>
	<a class="ipsLikeButton ipsLikeButton_disabled rep_down" href="adres" title="Nie lubię tego :(">
		Nie lubię
	</a>
</li>

a JS zmienia tylko style dla <li> na 

style="display: none;"

dla nieaktywnego <li> + dodaje

::before

dla aktywnego <li>.

więc cez tego w CSS takie coś wystarczy :

<li style="display:none">
	<a class="ipsLikeButton ipsLikeButton_enabled rep_up" href="adres" title="Lubię to!">
		✓ Lubię to
	</a>
</li>

No chyba że łatwiej będzie wam przeanalizować [co jest lepsze na dłuższą metę] prototype.js czy tak ipb.js

 

Ps. Nie wiem jak wy to zrobiliście ze zmianą koloru 'lubię to' ale kod dalej pobiera te zielone tabelki ale ich nie wczytuje.

 

• 

&

• 

Wszystko mało być zrobione z poziomu CSS, dlatego nic w szablonie nie zostało zmienione. Wiemy jak to rozwiązać, a analizowanie biblioteki JS do niczego nie jest potrzebne. No i chyba nieco się rozpędziłeś stwierdzając, że ta część szablonu nie korzysta z tłumaczeń.

[Milian]

Błędy trzeba naprawić. Tak. 

[3nichilat0]

@Miszczu

 

Co do tłumaczeń to mogłem przesadzić bo mogę jedynie zobaczyć lang.js ale:

 

 

////////Edit

 

To nie lepiej zrobić 1 porządne rozwiązanie?

 

Zmienić:

        .ipsLikeButton.ipsLikeButton_enabled {
                background: #2983B8;
                /*border: 1px solid #5ea128;*/
        } 

Na:

        .ipsLikeButton.ipsLikeButton_enabled {
                background: #2983b8 url('http://www.mpcforum.pl/public/style_images/def342/like_button.png') no-repeat 0;
                /*border: 1px solid #5ea128;*/
        }

A obrazek:

• 

  http://www.mpcforum.pl/public/style_images/def342/like_button.png

Zmienić na np:

• 

• http://www.mpcforum.pl/uploads/images/734671413279240638160.png
  

 

Czy takie rozwiązanie nie byłoby lepsze? 

 

To by zlikwidowało to ::Before a całość by była zbudowana na tej samej zasadzie jak 'Nie lubię' [w sensie bg przycisku]

[Gość]

Byłoby, ale obrazek również nie wchodził w grę, dlatego tak zostało to zrobione.

[Kubek]

Z względów estetycznych, tak

[HardStyl3r]

 

He, myślałem że tylko ja tak mam ;o

 

Jasne, że tak.

 

[Emo2]

Tak.

[TenStylOdLajka]

Taka poprawka, może się przydać, nie mam zastrzeżeń.

[Biter.]

Tak.

[wakc34]

Jak sie da wykonać to tak

[Mroczeks]

tak;c

[Haritirox]

Tak.

[Rayso.]

Ja mam wszystko dobrze. Niech będzie tak

[3nichilat0]

@Seldek IV

Bo już to naprawili. 

 

 

@Istny Diabel

Proponuje przenieść temat do rozpatrzonych gdyż naprawili już ten błąd.