actualbudget / actual

A local-first personal finance app
https://actualbudget.org
MIT License
12.58k stars 960 forks source link

:speech_balloon: (bank-sync) update sync button label to improve discoverability #2899

Closed MatissJanis closed 2 weeks ago

MatissJanis commented 2 weeks ago

We are getting a few confused folks on Discord that confuse the "sync" button at the top of the page with the "sync" button in account view.

Updating the label of this button to clear the confusion a bit.

netlify[bot] commented 2 weeks ago

Deploy Preview for actualbudget ready!

Name Link
Latest commit c2fa70dab0be8ef2276d09e53f8954864b0893ac
Latest deploy log https://app.netlify.com/sites/actualbudget/deploys/6671d1150235b90008d8f086
Deploy Preview https://deploy-preview-2899.demo.actualbudget.org
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 site configuration.

github-actions[bot] commented 2 weeks ago

Bundle Stats — desktop-client

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files count Total bundle size % Changed
9 4.75 MB → 4.75 MB (+10 B) +0.00%
Changeset File | Δ | Size ---- | - | ---- `src/components/accounts/Header.jsx` | 📈 +10 B (+0.06%) | 15.24 kB → 15.25 kB
View detailed bundle breakdown
**Added** No assets were added **Removed** No assets were removed **Bigger** Asset | File Size | % Changed ----- | --------- | --------- static/js/wide.js | 263.59 kB → 263.6 kB (+10 B) | +0.00% **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- static/js/resize-observer.js | 18.37 kB | 0% static/js/indexeddb-main-thread-worker-e59fee74.js | 13.5 kB | 0% static/js/BackgroundImage.js | 122.29 kB | 0% static/js/usePreviewTransactions.js | 790 B | 0% static/js/AppliedFilters.js | 20.25 kB | 0% static/js/narrow.js | 75.73 kB | 0% static/js/ReportRouter.js | 1.23 MB | 0% static/js/index.js | 3.01 MB | 0%
github-actions[bot] commented 2 weeks ago

Bundle Stats — loot-core

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files count Total bundle size % Changed
1 1.13 MB 0%

Changeset

No files were changed

View detailed bundle breakdown
**Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- kcab.worker.js | 1.13 MB | 0%
psybers commented 2 weeks ago

Definitely an improvement.

I still think the top-right server 'Sync' should hide under the server button. That would absolutely fix the problem, even without having to rename the bank sync button.

MatissJanis commented 2 weeks ago

I still think the top-right server 'Sync' should hide under the server button. That would absolutely fix the problem, even without having to rename the bank sync button.

That's an interesting proposal, but I can also see this being controversial. So lets go ahead with the low-hanging fruit for now and we can circle back to this later if the top sync button is still confusing folks.