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

trademark page is hard to find #959

Closed
peff opened this issue Feb 17, 2017 · 12 comments
Closed

trademark page is hard to find #959

peff opened this issue Feb 17, 2017 · 12 comments

Comments

@peff
Copy link
Member

peff commented Feb 17, 2017

The trademark policy at https://git-scm.com/trademark is a little hard to find just navigating site links. You have to get to https://git-scm.com/about/free-and-open-source. That is linked from the splash page, but in a reasonably subtle way. A more likely way is to hit the https://git-scm.com/about page. But then you have to realize that the circles are navigation elements (or click through all the way to the sixth page!).

We could probably improve this in two ways:

  1. The other sections of the site expand their subsections in the navbar, but "About" doesn't. It probably should.

  2. It's not immediately obvious that "Free and Open Source" is the right way to find out about the trademark. Perhaps the trademark should get its own mention on the navbar.

Patches welcome for either or both of those. I'd be open to other suggestions, too.

@akshatm147
Copy link
Contributor

@peff @pedrorijo91 Can you guide me how to solve the issue.

@pedrorijo91
Copy link
Member

Hey @chotuaashiq !

Thanks a lot for your enthusiasm in contributing :)

So, using the suggestions by @peff I would say :

  1. you should start by adding the capability of expanding the 'About' section on the navbar. This seems to be defined at app/views/shared/_sidebar.html.erb

  2. extracting the trademark page for it's own sub section on the nav bar should be about adding a new entry to the previous step

  3. You should think if we should add a new 'circle' menu item with the trademark page at the (https://git-scm.com/about/free-and-open-source)

These are just a few general hints (I wanted to give you some space to explore), but if you need any more details we can help you :)

@pedrorijo91
Copy link
Member

@chotuaashiq how's that going? Do you need any more guidance? let us know if you need anything :)

@akshatm147
Copy link
Contributor

@pedrorijo91 Still I am unable to figure out how to do this. Can you help?

@pedrorijo91
Copy link
Member

sure. just tell me a bit what have you tried, and what problems/doubts are you facing?

you should try to setup the application locally as described under https://github.com/git/git-scm.com#setup. Have you done this step?

you should have a look at the file https://github.com/git/git-scm.com/blob/master/app/views/shared/_sidebar.html.erb.

If you look at https://git-scm.com/doc and https://git-scm.com/about, you'll see the doc section expands, and the about no, right?

looking at the https://github.com/git/git-scm.com/blob/master/app/views/shared/_sidebar.html.erb you can see the /doc section has nested elements inside, while the /about section doesn't. So you probably need to add nested elements to the about section also

@sxlijin
Copy link
Contributor

sxlijin commented Dec 26, 2017

Mildly more concerning: why doesn't the trademark page show up when you search for "trademark"?

https://git-scm.com/search/results?search=trademark

@sxlijin
Copy link
Contributor

sxlijin commented Dec 26, 2017

I'm not sure we want an explicit section on the About page for this.

On the home page, the About page is described with the following tagline: "The advantages of Git compared to other source control systems."

Glancing at some other OSS projects:

From this alone I'm inclined to suggest simply either leaving it as is or adding a link in the footer.

@pedrorijo91
Copy link
Member

@sxlijin I understand your point. We could follow a similar approach of having the trademark linked on the footer. I can live with both solutions. But the solution on #1092 doesn't seem bad to me also. Besides that, after so much work, I think it would be a nice reward the author (a 1st time contributor), to see his changes going live.

@pedrorijo91
Copy link
Member

thinking about closing this issue since #1092 is already merged. If someone wants to revisit this, and possibly go for the solution of having the trademark in the footer, we can reopen it

@peff
Copy link
Member Author

peff commented Jan 3, 2018

why doesn't the trademark page show up when you search for "trademark"?

Because we only index the book and manpages, not the static content. That's probably fixable by adding a new rule to lib/tasks/search.rake, but we'd have to figure out the right incantation to feed the data to the indexer.

@pedrorijo91
Copy link
Member

maybe it's worth to create an issue to investigate indexing static content (trademark and other relevant pages) @peff ?

@peff
Copy link
Member Author

peff commented Jan 4, 2018

@pedrorijo91 Sure. I'm not planning to work on it myself, but somebody might want to pick it up. Opened #1109.

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

No branches or pull requests

4 participants