Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

Remove unused 'no UI' (iframe) mode #1071

Merged
merged 1 commit into from
Apr 26, 2021
Merged

Conversation

bbecquet
Copy link
Contributor

Description

Remove the 'no UI' mode, which was meant to be used as a quick and temporary solution for a legacy IA… but sadly never got used because modern IA got quickly implemented in the new search.
In practice it's no use to keep it. Even if we want an "embedded" mode at some point it will have different requirements than IA and we'll do it in a more robust way.
So just clean the code by removing these useless parts… and let's hope we'll be better at anticipating real needs next time.

Why

Dead code 😢 🧟

@bbecquet bbecquet requested review from xem and amatissart April 26, 2021 09:00
Copy link
Contributor

@xem xem left a comment

Choose a reason for hiding this comment

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

nice cleanup \o/

@bbecquet bbecquet merged commit bd1686e into Qwant:master Apr 26, 2021
@bbecquet bbecquet deleted the remove-no-ui branch April 26, 2021 12:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants