Skip to content

Fix compatibility with Thunderbird 143–147#772

Open
qbit-0 wants to merge 3 commits intojobisoft:masterfrom
qbit-0:master
Open

Fix compatibility with Thunderbird 143–147#772
qbit-0 wants to merge 3 commits intojobisoft:masterfrom
qbit-0:master

Conversation

@qbit-0
Copy link
Copy Markdown

@qbit-0 qbit-0 commented Jan 27, 2026

Summary

  • Fix hidden attribute handling for Thunderbird 143+: use removeAttribute("hidden") instead of setAttribute("hidden", false), which sets the attribute to the string "false" and doesn't actually unhide the element
  • Add --menuitem-icon CSS custom property for TB143+ menu icon rendering (TB switched from list-style-image)
  • Bump strict_max_version to 147.* and version to 4.16.1

Files changed

  • manifest.json — version 4.164.16.1, strict_max_version 140.*147.*
  • content/manager/accounts.js — fix hidden attribute handling (3 lines)
  • content/skin/ab.css — add --menuitem-icon property (2 lines)

Test plan

  • Tested on Thunderbird 147.0 (Mac and Windows)
  • TbSync account manager displays correctly (hidden attribute fix)
  • Menu icons render properly (CSS fix)
  • EAS provider sync works end-to-end

Servers tested

  • MailSite 10.4
  • Outlook.com
  • Office365.com

qbit-0 added 3 commits January 8, 2026 16:38
- Update strict_max_version to 146.*
- Fix hidden attribute handling: use removeAttribute("hidden") instead of
  setAttribute("hidden", false) which doesn't properly unhide elements in TB143+
- Add --menuitem-icon CSS property for TB143+ menu icon compatibility
@MaKaNu
Copy link
Copy Markdown

MaKaNu commented Feb 9, 2026

This fix worked on my deployment gist!

Tested on Linux with 145.0

@jim-davenport
Copy link
Copy Markdown

How do I install this?

@VisruthSK
Copy link
Copy Markdown

@jim-davenport Looks like someone made a fork and merged a similar fix. You could install their XPI: https://github.com/NielBuys/TbSync/releases/tag/v4.16.1

@jdiez17
Copy link
Copy Markdown

jdiez17 commented Feb 11, 2026

Works for me on Thunderbird 147, thank you!

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.

5 participants