Navigation Menu

Skip to content
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

Add ULS language selector & create global static folder #20

Merged
merged 3 commits into from Mar 2, 2015

Conversation

cydrobolt
Copy link
Contributor

  • Add static/global, builds to /static/
  • Add ULS language selector:
    • Remove old language selector
    • Add needed files to global static folder
    • Set up build script to work with ULS

@@ -45,16 +48,7 @@ <h3 class="masthead-brand">${title}</h3>
% endfor

<li class="dropdown">
Copy link
Contributor

Choose a reason for hiding this comment

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

This whole dropdown entry can probably be removed from the top navbar now that the uls language selector is in the footer, no?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops. You're right

@cydrobolt
Copy link
Contributor Author

Does the freeze affect this?

@ralphbean
Copy link
Contributor

Does the freeze affect this?

I don't think so. It (1) isn't hosted in infra and (2) doesn't play a role in preparing the alpha release, so I think we're free to make changes.

@nirik
Copy link
Member

nirik commented Mar 1, 2015

Yeah, no freeze concerns here... should be able to make whatever changes you need. ;)

@cydrobolt
Copy link
Contributor Author

@ralphbean I took away the unneeded dropdown. What do you think?

@ralphbean
Copy link
Contributor

@cydrobolt I had to make only one little change in 10938ef, but it looks good. Thanks for this!

It's live up on http://whatcanidoforfedora.org now 🚀

ralphbean added a commit that referenced this pull request Mar 2, 2015
Add ULS language selector & create global static folder
@ralphbean ralphbean merged commit 0f42aa7 into develop Mar 2, 2015
@ralphbean ralphbean deleted the uls-selector branch March 2, 2015 14:42
@cydrobolt
Copy link
Contributor Author

On http://whatcanidoforfedora.org, the global static files aren't in the correct place. Works locally, though. Maybe something with OpenShift? @ralphbean

@ralphbean
Copy link
Contributor

the global static files aren't in the correct place

Really?

  • How can you tell?
  • Where should they be?

Did you try doing a hard-reload of the page?

@cydrobolt
Copy link
Contributor Author

Yes, tried hard-reload. I know the global static files aren't correct because the ULS selector is missing images. Look in console

@ralphbean
Copy link
Contributor

A data point: it is looking for svgs, but the pngs are the only things there: http://whatcanidoforfedora.org/static/img/

@cydrobolt
Copy link
Contributor Author

Hmm. Shouldn't the SVGs be in there too?

@cydrobolt
Copy link
Contributor Author

Yeah, you're right. It looks like I'll have to add the SVGs back in.

@ralphbean
Copy link
Contributor

Hmm. Shouldn't the SVGs be in there too?

Ah ha! You never committed them. They are ignored in the .gitignore file. Can you add them explicitly and push to the uls-selector branch?

@cydrobolt
Copy link
Contributor Author

Mhm. Is there a reason why the SVGs are in the .gitignore?

@cydrobolt
Copy link
Contributor Author

Added in 1f6615e

@ralphbean
Copy link
Contributor

Mhm. Is there a reason why the SVGs are in the .gitignore?

To ignore the graphs produced by running asknot-ng.py --graph.

@ralphbean
Copy link
Contributor

It looks better now, but http://whatcanidoforfedora.org/static/img/world_map.svg is still missing.

@cydrobolt
Copy link
Contributor Author

Added it in 6b9ad8d
Apparently, it wasn't in the ULS source, so I had to dig it up from gf.o

@ralphbean
Copy link
Contributor

Looks good! Thanks @cydrobolt!

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.

None yet

3 participants