Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 8, 2025

Problem

The setup script wasn't updating the title in index.html when users accepted the default presentation title by pressing Enter. This occurred because:

  1. The default title was set to "Presentation Title" (line 40)
  2. The placeholder text in index.html was also "Presentation Title" (line 7)
  3. When sed tried to replace "Presentation Title" with "Presentation Title", no change occurred

As a result, users who accepted the default title ended up with a generic "Presentation Title" in their HTML instead of a meaningful title.

Solution

Changed the default presentation title from the static string "Presentation Title" to use the repository name ($REPO). Now when users press Enter to accept the default:

# Before
read -p "Enter presentation title [Presentation Title]: " TITLE
TITLE=${TITLE:-"Presentation Title"}

# After  
read -p "Enter presentation title [$REPO]: " TITLE
TITLE=${TITLE:-"$REPO"}

Examples

With git repository (e.g., my-awesome-talk):

  • Press Enter → title becomes "my-awesome-talk"
  • Enter custom text → title becomes the custom text

Without git repository:

  • Press Enter → title becomes "YOUR-REPO-NAME" (still a placeholder, but gets replaced)
  • Enter custom text → title becomes the custom text

This ensures the sed replacement always has an effect, properly updating both index.html and README.md with the chosen title.

Fixes #[issue-number]

Original prompt

This section details on the original issue you should resolve

<issue_title>The script doesn't update the title in the index.html</issue_title>
<issue_description>The title in the index.html should be updated by the template fixup script.</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #7

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Changed the default presentation title from "Presentation Title" to use the repository name. This ensures that when users press Enter to accept the default, the title in index.html actually gets updated instead of remaining as the placeholder "Presentation Title".

Co-authored-by: rfay <112444+rfay@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix title update issue in index.html Fix setup script to update index.html title when using default value Oct 8, 2025
Copilot AI requested a review from rfay October 8, 2025 19:09
@rfay rfay marked this pull request as ready for review October 8, 2025 19:34
@rfay rfay merged commit 3caa875 into main Oct 9, 2025
2 of 4 checks passed
@rfay rfay deleted the copilot/fix-index-html-title-update branch October 9, 2025 04:56
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.

The script doesn't update the title in the index.html

2 participants