Core 1.5.x #136

Merged
merged 1 commit into from Jan 9, 2013

Conversation

Projects
None yet
2 participants
Contributor

jesselong commented Jul 10, 2012

Handy class I use all the time, might save someone some boilerplate code.

@jesselong jesselong Add ReferenceLoadableDetachableModel - a LoadableDetachableModel that…
… uses a reference (think PK) to the object to load it.
c8f87cc

There are some differences in the indentation characters.
Other than that I think it is OK to merge it.

Owner

jesselong replied Jul 10, 2012

OK, for reference sake, what is the preferred wicket/wicketstuff indent scheme. I recently switched to using tabs for wicket stuff because that seems to be what is happening in wicket.

Is that right? tabs to indent?

Do you actually read it at 8 characters indented or do you configure your ide to display tabs at 4 char width?

I care just to be consistent - either spaces or tabs.
I think your file have them mixed. At least this is how GitHub shows them. It is not a big deal though. I'd merge it anyway.

My personal preference is tab. For Java my IDE shows them as 4 char width, for Scala - 2.

martin-g merged commit 371d571 into wicketstuff:core-1.5.x Jan 9, 2013

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