Removed name spacing #26

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@TMaYaD

TMaYaD commented Oct 25, 2011

It creates more trouble than it solves
Can not properly lookup the partials with namespacing

when we are using only a single conttroller, pages is as good as high_voltage.

Removed namespacing
It creates more trouble than it solves
Can not properly lookup the partials with namespacing
@mjankowski

This comment has been minimized.

Show comment Hide comment
@mjankowski

mjankowski Oct 25, 2011

Owner

Can you more specifically describe the problem you want to solve here?

Owner

mjankowski commented Oct 25, 2011

Can you more specifically describe the problem you want to solve here?

@TMaYaD

This comment has been minimized.

Show comment Hide comment
@TMaYaD

TMaYaD Oct 25, 2011

With name spacing, partials are being looked up in high_voltage/pages/ rather than within pages/ which is counter intuitive.

TMaYaD commented Oct 25, 2011

With name spacing, partials are being looked up in high_voltage/pages/ rather than within pages/ which is counter intuitive.

@mjankowski

This comment has been minimized.

Show comment Hide comment
@mjankowski

mjankowski Oct 25, 2011

Owner

That's a use case for subclassing the controller with a PagesController of your own.

I added a note to the README.

Owner

mjankowski commented Oct 25, 2011

That's a use case for subclassing the controller with a PagesController of your own.

I added a note to the README.

@mjankowski mjankowski closed this Oct 25, 2011

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