Skip to content

[CODE] Handle display of Classes with multiple super-classes#320

Closed
Dataliberate wants to merge 1 commit intoschemaorg:masterfrom
Dataliberate:master
Closed

[CODE] Handle display of Classes with multiple super-classes#320
Dataliberate wants to merge 1 commit intoschemaorg:masterfrom
Dataliberate:master

Conversation

@Dataliberate
Copy link
Contributor

  1. Make Thing appear at end of ‘Properties from’ listing
  2. Remove Inheritance display (breadcrums) from label display, replace with separate listing below for all inheritance path(s)

1. Make Thing appear at end of ‘Properties from’ listing
2. Remove Inheritance display (breadcrums) from label display, replace
with separate listing below for all inheritance path(s)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"crumbs" is more conventional spelling

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danbri
Copy link
Contributor

danbri commented Feb 5, 2015

I forgot the bit where I say "thanks!" first...

@Dataliberate
Copy link
Contributor Author

No problem - both with the thanks and the correcting to reflect the
convention. I should have gotten around to doing this much sooner,
however the founding of sdo-gozer seemed a good moment to pick.

On 5 February 2015 at 13:57, Dan Brickley [email protected] wrote:

I forgot the bit where I say "thanks!" first...


Reply to this email directly or view it on GitHub
#320 (comment).

Richard Wallis
Founder, Data Liberate
http://dataliberate.com
Tel: +44 (0)7767 886 005

Linkedin: http://www.linkedin.com/in/richardwallis
Skype: richard.wallis1
Twitter: @rjw

@Dataliberate
Copy link
Contributor Author

Superseded by pull #567

@danbri
Copy link
Contributor

danbri commented Jun 8, 2015

Thanks for the fix Richard. For review see http://sdo-ganymede.appspot.com/LocalBusiness and below

@Dataliberate
Copy link
Contributor Author

http://sdo-ganymede.appspot.com/AutoPartsStore

Looks impressive ?

~Richard

On 8 Jun 2015, at 16:33, Dan Brickley <[email protected]mailto:[email protected]> wrote:

Thanks for the fix Richard. For review see http://sdo-ganymede.appspot.com/LocalBusiness and below

Reply to this email directly or view it on GitHubhttps://github.com//pull/320#issuecomment-110045129.

@danbri
Copy link
Contributor

danbri commented Jun 8, 2015

It does look good! Though we'll have to be careful that combinations of multiple inheritance don't blow up and create screenfulls of these paths :)

@danbri
Copy link
Contributor

danbri commented Jun 8, 2015

/cc @mfhepp @vholland

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants