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

Update home.html #14

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jamescridland
Copy link
Contributor

Limit podcast episode titles to just one line. This looks most beautiful with patch-5.

Limit podcast episode titles to just one line
@valcanobacon
Copy link
Contributor

@jamescridland could you include a picture illustrating the change?

@jamescridland
Copy link
Contributor Author

Have done here: #15 (comment)

@valcanobacon
Copy link
Contributor

Are both of these PRs required for the feature? or is one optional? If they are both one feature perhaps one PR is better.

@jamescridland
Copy link
Contributor Author

I agree that one PR is better. I lack the skill on Github's web interface, which is all I used for this, to make changes to two files under one PR. Sorry.

@valcanobacon
Copy link
Contributor

No need to apologize. Makes sense if you are using the Web UI, I been working in Gitlab for the last year and have rotted my mind.

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.

2 participants