Skip to content

Conversation

@bunsenstraat
Copy link
Collaborator

@bunsenstraat bunsenstraat commented Sep 17, 2025

  • updated top bar
  • updated file explorer
  • fix bugs with terminal buttons
  • file watcher issues
  • fixed tracking with new tracking manager

@netlify
Copy link

netlify bot commented Sep 17, 2025

Deploy Preview for reliable-cocada-166884 ready!

Name Link
🔨 Latest commit 4771e05
🔍 Latest deploy log https://app.netlify.com/projects/reliable-cocada-166884/deploys/6900caf560dede0008a96117
😎 Deploy Preview https://deploy-preview-6385--reliable-cocada-166884.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@bunsenstraat bunsenstraat changed the title Topbarelectron Desktop bugs Sep 17, 2025
@bunsenstraat bunsenstraat requested a review from yann300 October 21, 2025 19:19
@bunsenstraat bunsenstraat self-assigned this Oct 21, 2025
@bunsenstraat bunsenstraat marked this pull request as ready for review October 21, 2025 19:20
@yann300 yann300 requested a review from ioedeveloper October 23, 2025 12:31
}

// Check if plugin is already activated
if (plugin.isActivated()) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe listening on the "activate" event emitted by the manager will be a better solution here.

    plugin.on('manager', 'activate', (plugin) => {
        ....
    })

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No that event will be gone

<HomeTabTitle />
<HomeTabRecentWorkspaces plugin={plugin} />
{!(platform === appPlatformTypes.desktop) ? <HomeTabRecentWorkspaces plugin={plugin} /> : <HomeTabRecentWorkspacesElectron plugin={plugin} />}
{/* {!(platform === appPlatformTypes.desktop) ? <HomeTabFile plugin={plugin} /> : <HomeTabFileElectron plugin={plugin}></HomeTabFileElectron>} */}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove unused comment

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

and its already merged

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@nx-cloud
Copy link

nx-cloud bot commented Oct 28, 2025

View your CI Pipeline Execution ↗ for commit 4771e05

Command Status Duration Result
nx build plugin_api --configuration=production ... ✅ Succeeded <1s View ↗
nx lint vyper ✅ Succeeded <1s View ↗
nx lint remixd ✅ Succeeded <1s View ↗
nx lint solhint ✅ Succeeded <1s View ↗
nx lint doc-gen ✅ Succeeded <1s View ↗
nx lint remix-ui ✅ Succeeded <1s View ↗
nx lint learneth ✅ Succeeded <1s View ↗
nx lint debugger ✅ Succeeded <1s View ↗
Additional runs (23) ✅ Succeeded ... View ↗

☁️ Nx Cloud last updated this comment at 2025-10-28 14:07:42 UTC

@ci-pr-comment
Copy link

ci-pr-comment bot commented Oct 28, 2025

✅ E2E tests passed (workflow: web)

Last run: Tue, Oct 28, 2025, 02:19 PM UTC

All tests are now passing! Previous failures have been resolved.

@bunsenstraat bunsenstraat merged commit 192aafd into master Oct 28, 2025
34 checks passed
@bunsenstraat bunsenstraat deleted the topbarelectron branch October 28, 2025 14:32
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.

3 participants