phpBB

Development Discussion Board

phpBB's testing ground of bleeding edge code
Advanced search

[RFC|Merged] Rounded corners via css

These requests for comments have lead to an implemented feature that has been successfully merged into the 3.1/Ascraeus branch. Everything listed in this forum will be available in phpBB 3.1.

Re: [RFC] Rounded corners via css

Postby wGEric » Thu Jan 19, 2012 9:14 pm

Make sure you look at the known issues for pie. There is one about serving the correct content type that will affect how it needs to be implemented. To make sure it actually works on every server you will probably have to use the php method.
Eric
wGEric
Registered User
 
Posts: 510
Joined: Wed Jun 11, 2003 2:07 am

Re: [RFC] Rounded corners via css

Postby Arty » Thu Jan 19, 2012 9:19 pm

TheKiller wrote:But maybe you should do the fallowing:
If Google Chrome Frame is not installed, you can direct your users to an installation page.

That is a bad idea. Forum should work for users, not force them to install some addons. Such approach will only annoy users, causing them to switch to other forum solutions.

wGEric wrote:Make sure you look at the known issues for pie. There is one about serving the correct content type that will affect how it needs to be implemented. To make sure it actually works on every server you will probably have to use the php method.

That's bad :(
Thanks for finding it.
Formerly known as CyberAlien.
User avatar
Arty
Styles Team
Styles Team
 
Posts: 802
Joined: Wed Mar 06, 2002 2:36 pm
Location: Mars

Re: [RFC] Rounded corners via css

Postby imkingdavid » Fri Jan 20, 2012 2:58 am

Arty wrote:
TheKiller wrote:But maybe you should do the fallowing:
If Google Chrome Frame is not installed, you can direct your users to an installation page.

That is a bad idea. Forum should work for users, not force them to install some addons. Such approach will only annoy users, causing them to switch to other forum solutions,

+1 phpBB should work out of the box without requiring users to go install even more things. Of course, depending on their browser that may mean a less than optimal browsing experience (although with IE, that's to be expected :lol:), but that's neither our fault nor our problem. We do what we can to make the experience as similar across browsers as possible, understanding that some browsers don't play as nicely as others. Major features should definitely take such things into account, even to the point of leaving them out if not all users will be able to use them, but something like rounded corners isn't a major feature. Besides, pointy corners aren't all bad, and the page load will decrease because the image won't have to be loaded each time.
I do custom MODs. PM for a quote!
View My: MODs | Portfolio
Please do NOT contact for support via PM or email.
Remember, the enemy's gate is down.
User avatar
imkingdavid
Development Team
Development Team
 
Posts: 905
Joined: Thu Jul 30, 2009 12:06 pm

Re: [RFC] Rounded corners via css

Postby Ger » Fri Jan 20, 2012 12:08 pm

imkingdavid wrote:+1 phpBB should work out of the box without requiring users to go install even more things. Of course, depending on their browser that may mean a less than optimal browsing experience (although with IE, that's to be expected :lol:), but that's neither our fault nor our problem. We do what we can to make the experience as similar across browsers as possible, understanding that some browsers don't play as nicely as others. Major features should definitely take such things into account, even to the point of leaving them out if not all users will be able to use them, but something like rounded corners isn't a major feature. Besides, pointy corners aren't all bad, and the page load will decrease because the image won't have to be loaded each time.
+1.
Above message may contain errors in grammar, spelling or wrongly chosen words. This is because English isn't my mother tongue. My apologies in advance.
User avatar
Ger
Registered User
 
Posts: 179
Joined: Mon Jul 26, 2010 1:55 pm
Location: 192.168.1.100

Re: [RFC] Rounded corners via css

Postby callumacrae » Fri Jan 20, 2012 5:06 pm

TheKiller wrote:But maybe you should do the fallowing:
If Google Chrome Frame is not installed, you can direct your users to an installation page.

That isn't phpBB's place to do, I would strongly disagree with it.


naderman wrote:I'd rather skip any IE specific workarounds like PIE, and just go with the CSS3 solution. Not having rounded corners is really not the end of the world.

+1
"In JavaScript, there is a beautiful, elegant, highly expressive language that is buried under a steaming pile of good intentions and blunders"
—Douglas Crockford

View my MOD, phpBB Mobile
User avatar
callumacrae
Website Team
Website Team
 
Posts: 883
Joined: Tue Apr 27, 2010 9:37 am
Location: England

Re: [RFC] Rounded corners via css

Postby Dragosvr92 » Fri Jan 20, 2012 7:17 pm

Well i say to just get on with the CSS3 propertys and ignore IE, Like Naderman said.
Its okay if they dont see round corners, not really a big issue. If they want performance they should upgrade their browsers, same like with operating systems. I guess the PIE Option could be made into a MODification and the forum owners can add it if they feel like it. The MOD Could have a module that enables the PIE Addon and a warning message to upgrade to a modern browser, or at least IE9. Or... add two options for PiE And Google Chromes frame..... Theres too many ifs.. i sort of get confused in writing :roll:
Previous username: TheKiller
Avatar on Memberlist 1.0.3
User avatar
Dragosvr92
Registered User
 
Posts: 358
Joined: Tue May 31, 2011 12:08 pm
Location: Romania

Re: [RFC] Rounded corners via css

Postby hanakin » Sat Jan 21, 2012 1:45 am

seeing as how i first proposed it obviously +1 lol :D also something to consider al be it not related to round corners but if we do away with all the gradients on the forums and the header and change the color scheme we can also eliminate the need for alot of the image overhead as well making the theme load faster more along the lines of subsilver, coupled with the dynamic sprite concept that was in the works if it ever gets implemented will completely streamline the theme. Only thing left from their as far as theme improvements go from the original topic is the HTML/CSS restructure into a cleaner and more semantic file. but really need to completely externalize the JS before that can/should happen
hanakin
Registered User
 
Posts: 176
Joined: Sat Dec 25, 2010 9:02 pm

Re: [RFC] Rounded corners via css

Postby Arty » Thu Mar 29, 2012 9:27 pm

Updated first post, created ticket.
Formerly known as CyberAlien.
User avatar
Arty
Styles Team
Styles Team
 
Posts: 802
Joined: Wed Mar 06, 2002 2:36 pm
Location: Mars

Re: [RFC|Merged] Rounded corners via css

Postby Arty » Tue Apr 10, 2012 7:59 pm

This RFC was solved by Shibu Lijack and merged.

PR: https://github.com/phpbb/phpbb3/pull/674
Formerly known as CyberAlien.
User avatar
Arty
Styles Team
Styles Team
 
Posts: 802
Joined: Wed Mar 06, 2002 2:36 pm
Location: Mars

Re: [RFC|Merged] Rounded corners via css

Postby Vinny » Sun May 06, 2012 4:01 pm

border-radius should be applied to docs/stylesheet.css too.


igorw wrote:As per IRC, images/corners_left.png and images/corners_right.png can be removed. @cyberalien mentioned that having to add that clear class everywhere is not so nice.

Any reason to keep the images corners_left2.gif and corners_right2.gif?
User avatar
Vinny
Styles Team
Styles Team
 
Posts: 92
Joined: Thu May 20, 2010 4:01 am
Location: Brazil

Previous Next

Return to [3.1/Ascraeus] Merged RFCs

Who is online

Users browsing this forum: Exabot [Bot], Google [Bot] and 8 guests