Skip to content

Ivan smirnov's redesign (2nd try) #1190

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 17 commits into from
Jan 17, 2016

Conversation

aG0aep6G
Copy link
Contributor

Do not merge. This breaks downloads.dlang.org.

Forum thread: http://forum.dlang.org/post/n6pdes$1upn$1@digitalmars.com

This being a revert of a revert, the little commit information that was there is gone. It can be seen over at #1187.

Reitering that someone should probably recreate the Twitter widget from a different account. #1187 (comment)

@andralex
Copy link
Member

@CyberShadow are you on this? Let me know if you need help. Thx!

reference, and what the article calls a $(I dynamic array)
is not named specifically by the language reference. Still,
the article is a great help in getting a good understanding
of D's array semantics.)
Copy link
Member

Choose a reason for hiding this comment

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

Unfortunately the language spec uses different terminology from this article ;)

@MartinNowak
Copy link
Member

Thanks, any ideas for downloads.dlang.org? We could operate w/ a stripped down version until we redesign it as well.

@aG0aep6G
Copy link
Contributor Author

I've made a companion PR for forum.dlang.org: CyberShadow/DFeed#51. That needs some changes to dlang.org's forum-template.dd. I've added them here.

@aG0aep6G
Copy link
Contributor Author

Thanks, any ideas for downloads.dlang.org? We could operate w/ a stripped down version until we redesign it as well.

Where is the git repository of downloads.dlang.org? Updating forum.dlang.org went pretty smoothly, so maybe I can just do downloads.dlang.org over the next couple days, and then we can make the switch for everyone at the same time.

Alternatively, we can freeze downloads.dlang.org in the old design by using copies of the dlang.org files (via git submodule) instead of using the online files. This may be a good idea anyway so that we don't accidentally break things again.

@CyberShadow
Copy link
Member

As an option I guess we could leave style.css etc. alone, and use new filenames for files used in the new design. But of course we should update it instead if possible.

So I don't know these things either:

  • Where is the source code for the downloads.dlang.org page templates?
  • How can we build and preview it?
  • Who can update it and push it to the live site?

@MartinNowak
Copy link
Member

Unfortunately it's not possible to generate the pages w/o an S3 account.
https://github.com/MartinNowak/downloads.dlang.org
Please don't put too much effort into this b/c we have to come up w/ something else in order to integrate the nightlies. I'd say updating the template so it's operable should be good enough for now.

@CyberShadow
Copy link
Member

Unfortunately it's not possible to generate the pages w/o an S3 account.

Doesn't look so bad, maybe a second main inside version(test) which supplies dummy data to buildIndex.

@MartinNowak
Copy link
Member

Doesn't look so bad, maybe a second main inside version(test) which supplies dummy data to buildIndex.

I can just do that, so you guys can work on the template.

<!-- Github ribbon -->
<a href="http://github.com/D-Programming-Language"><img id="github-ribbon"
src="images/github-ribbon.png" alt="Fork D on GitHub"></a>

Copy link
Member

Choose a reason for hiding this comment

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

Since the GitHub ribbon is removed, I think there should be a link to the source code somewhere (e.g. near the release archive / GPG keys links).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't think we need a link to GitHub on the downloads page, but here you go. Added a commit: aG0aep6G@2d1d95d

@CyberShadow
Copy link
Member

This seems to break building the CHM. I'll look into it.

@MartinNowak
Copy link
Member

I can just do that, so you guys can work on the template.

There you go MartinNowak/downloads.dlang.org@eed9a51...MartinNowak:9bd901c95681ce827da46fcd5aacc708266f953b, now generates html from a json file.

git clone git://github.com/MartinNowak/downloads.dlang.org.git
cd downloads.dlang.org
make -C src
cp test_index.json index.json
./src/build-gen-index generate

Not sure if it works on Windows though (it's using / to construct paths everywhere).

@CyberShadow
Copy link
Member

How about a changelog SUBNAV?

@CyberShadow
Copy link
Member

This seems to break building the CHM. I'll look into it.

aG0aep6G#2

@CyberShadow
Copy link
Member

What's with the link to the non-existing /menu.html ?

@CyberShadow
Copy link
Member

Main page screenshot with restyled forum widgets:

screen shot 2016-01-13 at 06 32 29

@MartinNowak
Copy link
Member

Nice

@CyberShadow
Copy link
Member

The D logo is broken in IE 11 (only top right corner is shown), maybe the SVG needs to be "resized".

@aG0aep6G
Copy link
Contributor Author

What's with the link to the non-existing /menu.html ?

/menu.html should be there. It's a no-JS fallback page.

menu.dd is here: https://github.com/D-Programming-Language/dlang.org/pull/1190/files#diff-39b0dfe44d7b515b991a96b903b95f9b

And here it is in posix.mak: https://github.com/D-Programming-Language/dlang.org/pull/1190/files#diff-228b518f4545d060f900578830ade5daR172

@aG0aep6G
Copy link
Contributor Author

Incorporated @CyberShadow's CHM fixes. Can't say that I have reviewed them. I don't know a thing about the CHM stuff.

@aG0aep6G
Copy link
Contributor Author

How about a changelog SUBNAV?

Added: aG0aep6G@43ca3df

@CyberShadow
Copy link
Member

/menu.html should be there. It's a no-JS fallback page.

Hm, OK. Not sure what I did to get a 404, but I can see it's all there.

Added: aG0aep6G/dlang.org@43ca3df

Nice!

@aG0aep6G aG0aep6G force-pushed the Ivan-Smirnov's-redesign branch from 427c257 to e013cf7 Compare January 13, 2016 23:29
@CyberShadow
Copy link
Member

Incorporated @CyberShadow's CHM fixes. Can't say that I have reviewed them. I don't know a thing about the CHM stuff.

Most of it has been shuffling the navigation macros to allow emitting a complete navigation tree in a JSON-ish format that chmgen can parse.

@CyberShadow
Copy link
Member

@aG0aep6G Will you be looking into updating the downloads.dlang.org template? That currently has the highest priority.

…ert-1187-Ivan-Smirnov's-redesign"

This reverts commit b6fddab, reversing
changes made to 910c4f6.
@aG0aep6G aG0aep6G force-pushed the Ivan-Smirnov's-redesign branch from e013cf7 to 040069d Compare January 14, 2016 13:23
@aG0aep6G
Copy link
Contributor Author

Pull request for downloads.dlang.org - quick and dirty, not making it more robust against future breakage: https://github.com/MartinNowak/downloads.dlang.org/pull/1

The Visual D pages need to put a non-ROOT_DIR path there.
@aG0aep6G
Copy link
Contributor Author

And here's a PR for the Visual D pages: dlang/visuald#57

@aG0aep6G
Copy link
Contributor Author

The D logo is broken in IE 11 (only top right corner is shown), maybe the SVG needs to be "resized".

Added a commit to fix that, and one to fix the search dropdown style.

@andralex
Copy link
Member

Thanks all for the hard work! @CyberShadow @MartinNowak are we on track to push the bits to dlang.org on Friday (essentially 12 hours from now)?

@CyberShadow
Copy link
Member

Blockers:

  • downloads.dlang.org (waiting on Martin)

Non-blocking issues:

Possible issues (needs more discussion):

  • Change the web font (or not) to a standard one
  • Make the gray color used in code comments and other areas darker

Looks a bit nicer with the sidebar. Also helps forum.dlang.org with height
calculations.
Also fix a subtle alignment issue that would show when expanding/collapsing
an item in the phobos subnav.
@aG0aep6G aG0aep6G force-pushed the Ivan-Smirnov's-redesign branch from ba0594a to 387922f Compare January 15, 2016 20:05
@aG0aep6G
Copy link
Contributor Author

Added two commits:

@MartinNowak
Copy link
Member

downloads.dlang.org (waiting on Martin)

Merged https://github.com/MartinNowak/downloads.dlang.org/pull/1, anything else?

@CyberShadow
Copy link
Member

Merged MartinNowak/downloads.dlang.org#1, anything else?

I guess no, we just need to deploy this PR and downloads.dlang.org in tandem.

@andralex
Copy link
Member

@CyberShadow wanna do the honors?

@CyberShadow
Copy link
Member

Well, I think @MartinNowak should do that since I can't update downloads.dlang.org.

@andralex
Copy link
Member

@MartinNowak you got the red button

Safari seems to have problems with word-spacing.
@aG0aep6G
Copy link
Contributor Author

Added a last minute fix as per CyberShadow/DFeed#52 (comment)

@MartinNowak
Copy link
Member

Will deploy this in an hour.

MartinNowak added a commit that referenced this pull request Jan 17, 2016
@MartinNowak MartinNowak merged commit f23f367 into dlang:master Jan 17, 2016
@MartinNowak
Copy link
Member

Will deploy this in an hour.

Done, don't forget to press F5.

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.

6 participants