Yii Framework Forum: Should we keep the skin feature of widgets - Yii Framework Forum

Jump to content

Page 1 of 1
  • You cannot start a new topic
  • You cannot reply to this topic

Should we keep the skin feature of widgets

Poll: Should we keep the skin feature of widgets (57 member(s) have cast votes)

Should we keep the skin feature of widgets

  1. Yes (7 votes [12.28%])

    Percentage of vote: 12.28%

  2. No (50 votes [87.72%])

    Percentage of vote: 87.72%

Vote Guests cannot vote

#1 User is offline   qiang 

  • Yii Project Lead
  • Yii
  • Group: Yii Dev Team
  • Posts: 5,871
  • Joined: 04-October 08
  • Location:DC, USA

Posted 17 May 2012 - 06:18 AM

Skin usage: http://www.yiiframew...cs.theming#skin

Should we keep this feature in 2.0? If few people are using this feature, I'm thinking to drop this feature for better performance.
0

#2 User is offline   CeBe 

  • Advanced Member
  • Yii
  • Group: Yii Dev Team
  • Posts: 469
  • Joined: 16-July 10
  • Location:Berlin. Germany

Posted 17 May 2012 - 08:21 AM

I'd rather say we should merge skins and themes so that there is only one thing that allows overwriting/replacing views.
So if CTheme allows theming of widget views, we could leave the skins behind.
0

#3 User is offline   Haensel 

  • Advanced Member
  • PipPipPip
  • Yii
  • Group: Members
  • Posts: 444
  • Joined: 14-January 11
  • Location:Vienna (Austria)

Posted 17 May 2012 - 08:35 AM

Agree, I've never used widget skins either
0

#4 User is offline   Da:Sourcerer 

  • Master Member
  • PipPipPipPip
  • Yii
  • Group: Members
  • Posts: 1,193
  • Joined: 30-March 11
  • Location:Berlin, Germany

Posted 17 May 2012 - 10:08 AM

+1 by me. I found it a neat feature but never actually used it.
programmer /ˈprəʊgramə/, noun: a device that converts ►coffee into ►code
0

#5 User is offline   yiqing95 

  • Advanced Member
  • PipPipPip
  • Yii
  • Group: Members
  • Posts: 586
  • Joined: 27-December 10
  • Location:china

Posted 17 May 2012 - 10:14 AM

me too , never used it :lol:
0

#6 User is offline   Junior - df9 

  • Advanced Member
  • PipPipPip
  • Yii
  • Group: Members
  • Posts: 415
  • Joined: 24-May 09
  • Location:Brazil

Posted 17 May 2012 - 11:04 AM

I thing that as long as we have a way to customize the looking of widgets inside a theme, no problem removing that. I mean, if Yii looks inside themes/themename/components/views to render the widget view, cannot see why to keep the feature.

To sum it up, I have also never used it and performance matters too much, agreed removing feature! :)

Regards
______________________________________
Junior
df9.com.br
Linux Registered User #364954
GNU/Linux: together we're ready!
0

#7 User is offline   twisted1919 

  • Master Member
  • PipPipPipPip
  • Yii
  • Group: Members
  • Posts: 617
  • Joined: 23-October 10
  • Location:Romania

Posted 17 May 2012 - 02:04 PM

I use widgets too much in my projects, but i never used the skins feature, so i think it's safe to be removed.
0

#8 User is offline   Psih 

  • Standard Member
  • PipPip
  • Yii
  • Group: Members
  • Posts: 114
  • Joined: 30-June 10

Posted 17 May 2012 - 03:37 PM

I think that the same functionality is easily achievable inside a widget with an additional param telling witch view to render, but that's quite rare and tends to be specific functionality for specific cases. So just get rid of it - "I have a need, a need for speed" © Top Gun
0

#9 User is offline   yugene 

  • Advanced Member
  • PipPipPip
  • Yii
  • Group: Members
  • Posts: 492
  • Joined: 08-August 09

Posted 18 May 2012 - 02:02 AM

+1
for never used it :)
0

#10 User is offline   Seal 

  • Standard Member
  • PipPip
  • Yii
  • Group: Members
  • Posts: 126
  • Joined: 02-February 10

Posted 18 May 2012 - 02:09 AM

It's funny, I never used theme either. I was put off from the reality that skin may degrade performance.Posted Image

@CeBe - I like the suggestion of merging skin and themes.

@Qiang, is this a possible?
Sylvester La-Tunje

Posted Image
0

#11 User is offline   Dart 

  • Newbie
  • Yii
  • Group: Members
  • Posts: 17
  • Joined: 03-November 10

Posted 19 May 2012 - 06:06 PM

Here's a yes vote. I use skins all the time.

This post has been edited by Dart: 20 May 2012 - 04:14 PM

0

#12 User is offline   Narretz 

  • Junior Member
  • Pip
  • Yii
  • Group: Members
  • Posts: 48
  • Joined: 20-August 11

Posted 20 May 2012 - 12:21 PM

View PostDart, on 19 May 2012 - 06:06 PM, said:

Here's a no vote. I use skins all the time.


I guess you mean a "yes" vote. Or are skins so horrible that you want to get rid of them?
0

#13 User is offline   Dart 

  • Newbie
  • Yii
  • Group: Members
  • Posts: 17
  • Joined: 03-November 10

Posted 20 May 2012 - 04:25 PM

View PostNarretz, on 20 May 2012 - 12:21 PM, said:

I guess you mean a "yes" vote. Or are skins so horrible that you want to get rid of them?


Edited. I use them to set global parameters for widgets, especially themeUrl and theme.
0

#14 User is offline   yugene 

  • Advanced Member
  • PipPipPip
  • Yii
  • Group: Members
  • Posts: 492
  • Joined: 08-August 09

Posted 23 June 2012 - 05:56 AM

I never used skins before as I stated in my previous comment, but they are used in the project I started to work on recently, so I'd like to add a bit changed opinion :)

I should admit, it's a very convenient feature in some cases, though, if it degrades performance significantly, I still think it's better to get rid of it, at least in such implementation.
Also, occasionally I found this really nice article by Dana Luther, which describes possible skin usages. This article made me to think one more time if there's definitly no need in skins :).
0

#15 User is offline   Antonio Ramirez 

  • Elite Member
  • Yii
  • Group: Yii Dev Team
  • Posts: 1,448
  • Joined: 04-October 10

Posted 27 June 2012 - 12:13 PM

Never used...
¿How long would it take for you to understand that you own nothing in this world?

www.ramirezcobos.com
www.2amigos.us
www.github.com/tonydspaniard
www.github.com/2amigos


Posted Image
0

Share this topic:


Page 1 of 1
  • You cannot start a new topic
  • You cannot reply to this topic

1 User(s) are reading this topic
0 members, 1 guests, 0 anonymous users