[BUG]: Direct DOM manipulation can lead to inconsistencies.

[aniket866]

Bug Description

• Where?: InvoicePreview.jsx ,CountryPicker.jsx, ReceivedInvoice.jsx

• Issue: Direct DOM manipulation: e.target.style.display = "none".

• Fix: Use React state (const [imgError, setImgError] = useState(false)) to handle image fallbacks.

• Why?: Bypassing React's virtual DOM can lead to inconsistencies.

Steps to Reproduce

N/A

Logs and Screenshots

N/A

Environment Details

N/A

Impact

Critical - Application is unusable

Code of Conduct

• I have joined the Discord server and will post updates there
• I have searched existing issues to avoid duplicates

[aniket866]

@kumawatkaran523 Please assign

[sach2004]

@DengreSarthak @kumawatkaran523 it is Valid. the e.target.style.display = "none" and e.target.src = fallback patterns are not standard practice for react. The src replacement one can actually cause infinite onError loops if the fallback also fails.