[RFC|Merged] Removal of imagesets

[Arty]

What do you mean?

[nextgen]

What do you mean?

This?

the css classes at the stylesheet root/styles/prosilver/theme/en/stylesheet.css are wrong :
This :
.imageset.button_topic_new {
Should ber :
.post-icon span {

because there are no call to any class named .imageset or .button_topic_new in the template, if you duplicate the file root/styles/prosilver/theme/en/stylesheet.css to another language and put your images inside that folder are never used, at least for big buttons and post buttons in prosilver style.

This is a fixed example for Spanish (es) language.

Code: Select all

/* Set big button dimensions */
.buttons div.reply-icon		{ width: 96px; height: 25px; }
.buttons div.post-icon		{ width: 96px; height: 25px; }
.buttons div.locked-icon	{ width: 88px; height: 25px; }
.buttons div.pmreply-icon	{ width: 96px; height: 25px; }
.buttons div.newpm-icon		{ width: 84px; height: 25px; }
.buttons div.forwardpm-icon	{ width: 96px; height: 25px; }

/* Set profile icon dimensions */
ul.profile-icons li.pm-icon		{ width: 28px; height: 20px; }
ul.profile-icons li.quote-icon	{ width: 54px; height: 20px; }
ul.profile-icons li.edit-icon	{ width: 42px; height: 20px; }

 /* ES Language Pack */
.pm-icon, .pm-icon a {
    background-image: url("./icon_contact_pm.gif");
}
.edit-icon, .edit-icon a {
	background-image: url("./icon_post_edit.gif");
}
.quote-icon, .quote-icon a {
	background-image: url("./icon_post_quote.gif");
}
.forwardpm-icon span {
    background-image: url("./button_pm_forward.gif");
}
.newpm-icon span {
    background-image: url("./button_pm_new.gif");
}
.pmreply-icon span {
    background-image: url("./button_pm_reply.gif");
}
.online {
    background-image: url("./icon_user_online.gif");
}
.locked-icon span {
    background-image: url("./button_topic_locked.gif");
}
.post-icon span {
	background-image: url("./button_topic_new.gif");
}
.reply-icon span {
	background-image: url("./button_topic_reply.gif");
}

[Arty]

You have outdated stylesheet.css. Look in recent develop branch.

[nextgen]

You have outdated stylesheet.css. Look in recent develop branch.

sure?
I got it for your profile on github
https://github.com/phpbb/phpbb3/blob/57 ... esheet.css

[Arty]

That's wrong URL. It points to old commit, not to latest version.
Here is correct URL: https://github.com/phpbb/phpbb3/blob/de ... esheet.css

[nextgen]

I said nothing then

PD: I guess for another language is not the same file?

[Arty]

File is the same, directory is different.

[nextgen]

File is the same, directory is different.

It solved most of my doubts as to change the imageset
so glad to collaborate with the project

[oddfish]

Take a look at very common situation: user has installed his language pack, but style does not have imageset for it.

In 3.0: edit, quote and other language specific images disappear
In 3.1: edit, quote and other language specific images fall back to English

That's why that change is there.

Appreciate that and I see this reason.
My issue is having different values to doing the same job. I thought the idea was to simplify and clean up the prosilver css, not to confuse it further by creating differences. That said, I should look closer at the template system calling the language images. Surely it can be simpler.

[Arty]

If some entries are the same, you can remove them from your custom language pack.