Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 31, 2025

Removes the 2-second loading screen that displayed before the webview loads.

Changes

  • Removed loading screen HTML (logo, title, spinner, message)
  • Removed setTimeout-based fade-out logic
  • Removed 144 lines of loading screen CSS and animations (fadeInUp, pulse, shimmer, bounce)

The webview now renders immediately without intermediate UI.

Original prompt

Remove the loading screen


💡 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.


Note

Removes the intermediate loading UI so the app shows the webview immediately.

  • Deletes loading-screen markup and related setTimeout fade logic from index.html; retains webview and dom-ready listener
  • Removes loading screen CSS and animation rules from styles.css; keeps frame and webview styling intact
  • No changes to CSP, icons, or webview source — only the pre-load UI was removed

Written by Cursor Bugbot for commit 520dadc. Configure here.

Copilot AI changed the title [WIP] Remove the loading screen Remove loading screen Dec 31, 2025
@sumitduster-iMac sumitduster-iMac marked this pull request as ready for review December 31, 2025 08:53
Copilot AI review requested due to automatic review settings December 31, 2025 08:53
@sumitduster-iMac sumitduster-iMac merged commit 1f66797 into main Dec 31, 2025
4 of 6 checks passed
@sumitduster-iMac sumitduster-iMac deleted the copilot/remove-loading-screen branch December 31, 2025 08:54
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the loading screen that displayed for 2 seconds before the webview rendered, allowing the application to render immediately on startup.

Key Changes:

  • Removed 144 lines of loading screen CSS including animations (fadeInUp, pulse, shimmer, bounce)
  • Removed loading screen HTML markup and associated JavaScript logic
  • Eliminated the 2-second setTimeout delay before showing the main webview

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
styles.css Removed all loading screen CSS classes (.loading-screen, .loading-content, .logo-container, etc.) and animation keyframes
index.html Removed loading screen HTML div, removed JavaScript that controlled the fade-out animation and display toggle

The code changes are clean and complete. All loading screen references have been successfully removed from the code files with no orphaned references remaining. The webview will now display immediately on application launch.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

<h1 class="loading-title">Lovable</h1>
<div class="loading-spinner">
<div class="spinner-dot"></div>
<div class="spinner-dot"></div>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI will fail: loading-screen check breaks

Removing the loading-screen element from index.html will cause the CI pipeline to fail. The workflow at .github/workflows/ci.yml line 54 runs grep -q 'loading-screen' index.html || exit 1 to verify the loading screen exists. With the loading screen removed, this check will fail and block the PR from merging.

Fix in Cursor Fix in Web

background: linear-gradient(135deg, #0F172A 0%, #1E293B 100%);
}

/* Loading Screen */
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI will fail: @Keyframes check breaks

Removing all @keyframes animation rules from styles.css will cause the CI pipeline to fail. The workflow at .github/workflows/ci.yml line 61 runs grep -q '@keyframes' styles.css || exit 1 to verify keyframe animations exist. With all keyframes removed, this check will fail and block the PR from merging.

Fix in Cursor Fix in Web

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