Skip to content

Fix #550 - Use dlang.org theme on the editor page#553

Merged
wilzbach merged 1 commit intodlang-tour:masterfrom
wilzbach:fix-550
Jul 2, 2017
Merged

Fix #550 - Use dlang.org theme on the editor page#553
wilzbach merged 1 commit intodlang-tour:masterfrom
wilzbach:fix-550

Conversation

@wilzbach
Copy link
Copy Markdown
Member

Instead of just dumping the menu, I went the "long" road and build the menu statically in D.
This should allow us more flexibility in the future.

content: " \f0d7";
font-family: FontAwesome;
}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Backported from dlang.org

Comment thread public/static/js/menu.js

if (container.className.indexOf("expand-container")) {
return;
}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

this is required, s.t. the link clicks on "Downloads" and "Packages" (i.e. no sections) don't get silently swallowed.

@wilzbach
Copy link
Copy Markdown
Member Author

A few screenshots:

image

image

Mobile

image

image

@CyberShadow
Copy link
Copy Markdown

I went the "long" road and build the menu statically in D

That's a good idea only if you want to maintain a third copy of the menu...

I'd be inclined to say that this is a better idea:
https://github.com/dlang/dlang.org/blob/master/forum-template.dd

@wilzbach
Copy link
Copy Markdown
Member Author

That's a good idea only if you want to maintain a third copy of the menu...

I think there are more copies out there: blog.dlang.org, code.dlang.org, VisualD, ...

I'd be inclined to say that this is a better idea:

Maybe in theory, but we already have adopted and trimmed down the menu CSS to a single CSS file and we need to maintain the ability to generate the menu dynamically anyways for the language content.
Also except for the menu CSS there's nothing taken over from dlang.org.

@CyberShadow
Copy link
Copy Markdown

I think there are more copies out there: blog.dlang.org, code.dlang.org, VisualD, ...

More things to fix, then.

The next time a major website redesign comes about, we are going to be neck deep into doing pointless work updating all these websites that could have been avoided with a bit of fore-planning and refactoring.

Except forum.dlang.org, of course.

Maybe in theory, but we already have adopted and trimmed down the menu CSS to a single CSS file and we need to maintain the ability to generate the menu dynamically anyways for the language content.

Instead of working around things in an unmaintainable manner on your side, please refactor things on the dlang.org side to maximize reuse.

Comment thread source/webinterface.d
toc, title, githubRepo, translations)();
}

private static auto buildDlangToc()
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks so much better than all those DDoc shenanigans at dlang.org!

@wilzbach
Copy link
Copy Markdown
Member Author

The next time a major website redesign comes about, we are going to be neck deep into doing pointless work updating all these websites that could have been avoided with a bit of fore-planning and refactoring.

Btw it's much more likely that we redesign the tour in the meanwhile, e.g. #473 ;-)

@PetarKirov
Copy link
Copy Markdown
Member

Instead of working around things in an unmaintainable manner on your side, please refactor things on the dlang.org side to maximize reuse.

The sooner we move dlang.org to vibe.d diet templates, the sooner this problem would be resolved :P

@wilzbach
Copy link
Copy Markdown
Member Author

The sooner we move dlang.org to vibe.d diet templates, the sooner this problem would be resolved :P

FWIW https://github.com/dlang/dlang.org/blob/master/dpl-docs/views/layout.dt

@wilzbach
Copy link
Copy Markdown
Member Author

wilzbach commented Jul 2, 2017

@CyberShadow I think it would be an unreasonable amount of work to import the the dlang.org tour Ddoc templete. Also we can't use submodules here, because sadly DUB still doesn't support them and the tour can be run offline (well currently this only works if you have the language translation, but it's an existing feature).

Lastly as mentioned before we/I plan to redesign the DTour a bit in the next few weeks, so we most likely are just going to keep the color schemes and style of dlang.org, which is only CSS anyhow.

@wilzbach wilzbach merged commit 60cb2f6 into dlang-tour:master Jul 2, 2017
@wilzbach wilzbach deleted the fix-550 branch July 2, 2017 14:24
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