Skip to content
This repository has been archived by the owner on May 3, 2018. It is now read-only.

scratchwikiskin #24

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

scratchwikiskin #24

wants to merge 15 commits into from

Conversation

Krett12
Copy link

@Krett12 Krett12 commented Aug 25, 2014

Please update!

Please update!
@maxamillion40
Copy link
Contributor

What does this do?

@seotts
Copy link
Contributor

seotts commented Aug 27, 2014

Yep, I gotta know what changes you made first.

@Krett12
Copy link
Author

Krett12 commented Aug 30, 2014

I commented it on your profile

Date: Wed, 27 Aug 2014 08:31:17 -0700
From: [email protected]
To: [email protected]
CC: [email protected]
Subject: Re: [scratchwikiskin] scratchwikiskin (#24)

Yep, I gotta know what changes you made first.


Reply to this email directly or view it on GitHub.

@seotts
Copy link
Contributor

seotts commented Sep 5, 2014

It's an interesting idea to bold the Scratch Wiki link in the footer, but I'm not sure we need it. The other links in the footer don't go bold when you visit them.

@Krett12
Copy link
Author

Krett12 commented Sep 6, 2014

Technically I don't think that the Wiki should look like Scratch at all (I don't even use ScratchWikiSkin (contributors can do that) ). But maybe we could update the forum skin, but really I don't think it should stay a link. But hey, I wanted to remove the link in the first place, but nobody liked that. Like how on the Scratch Wiki (or any wiki) attempting to link to what you are already on will turn to bold =

@maxamillion40
Copy link
Contributor

I don't use the ScratchWikiSkin either but I still care.

@maxamillion40
Copy link
Contributor

That would mean that scmb1 or another ST member would have to go change the code for every page that links in the footer. That would be 17 pages. That is way too much work. Sorry Krett.

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

Successfully merging this pull request may close these issues.

4 participants