-
Notifications
You must be signed in to change notification settings - Fork 63
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 rss icon #37
base: master
Are you sure you want to change the base?
Add rss icon #37
Conversation
It's a nice icon. I wonder how it would be used on svelte.dev. Perhaps we should see that to make sure we agree it's the best way to represent it and it will actually be used before we merge it here |
Been wondering the same, maybe we can add a footer since the site lacks it or we can add along with the header icons (github, discord). |
Vuejs news has them on every article like this; a static sidebar |
I was thinking we could put it at the bottom of the blog listings page like so. (source code) I like what @peopledrivemecrazy posted and think it would be great for the blog section to have a thing like that, but that would require Svelte to have a newsletter subscribe thing first. I'm a bit hesitant about putting it in the header of every page since it isn't exactly the best interface were someone to click it (just downloads an xml file to their computer) |
That would make one scroll to the end of the page to just get the RSS feed
link, well let's get more suggestions into this.
…On Wed, Jan 27, 2021 at 11:18 AM J Delaney ***@***.***> wrote:
@benmccann <https://github.com/benmccann>
[image: Screen Shot 2021-01-26 at 9 42 35 PM]
<https://user-images.githubusercontent.com/3589480/105948283-882f8380-601f-11eb-8fff-650ce8c70f4e.png>
I was thinking we could put it at the bottom of the blog listings page
like so. (source code
<sveltejs/svelte@master...j-delaney:site-rss-link>
)
I like what @peopledrivemecrazy <https://github.com/peopledrivemecrazy>
posted and think it would be great for the blog section to have a thing
like that, but that would require Svelte to have a newsletter subscribe
thing first. I'm a bit hesitant about putting it in the header of every
page since it isn't exactly the best interface were someone to click it
(just downloads an xml file to their computer)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#37 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA55UJKLFKVFLHFCTTSLO23S36SMHANCNFSM4WUTE7LA>
.
|
Seems like a good starting point. My main suggestion would be to make the whitespace between the last article and RSS link the same as the whitespace between articles |
@benmccann Great idea! Here's what it looks like now: I updated my branch as well. I have a question about CSS best practices I'd love advice on. In my branch I set the RSS's div to have a margin-top to match what the .rss {
margin-top: 3.2rem;
}
...
h2 {
display: inline-block;
margin: 3.2rem 0 0.4rem 0;
... Would it be better to put this into a variable or is it okay to hardcode it since it's only be used in 2 places (in the same file)? |
@benmccann This is a minor detail, but I like the icon more when the Icon itself has With What do you think about having that be controlled by a parameter to |
FYI, we're moving this repo to https://github.com/sveltejs/sites/tree/master/packages/site-kit. That location is currently used only for the SvelteKit docs, but we hope to eventually use it for all docs |
Summary
Adds an RSS icon to the icon library taken from from https://feathericons.com/?query=rss
Motivation
As discussed in Discord, so that we can add an
<Icon name="rss"> RSS Feed
to the bottom of https://svelte.dev/blog. I figured it would be helpful for people who need a direct link to the RSS feed so they don't need to go looking through the page sourceTest plan
Tested on the REPL: You can see this in action at https://svelte.dev/repl/1f1ae8d9d155435f82549f97d9d36a9c?version=3.32.0