Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Feautre tips view" #477

Merged
merged 1 commit into from
Mar 30, 2024
Merged

Revert "Feautre tips view" #477

merged 1 commit into from
Mar 30, 2024

Conversation

tisfeng
Copy link
Owner

@tisfeng tisfeng commented Mar 30, 2024

Reverts #418

@tisfeng tisfeng requested a review from AkaShark March 30, 2024 13:09
@tisfeng
Copy link
Owner Author

tisfeng commented Mar 30, 2024

This feature is often triggered by mistake #452 , so let's revert it and fix it later.

@tisfeng tisfeng requested a review from phlpsong March 30, 2024 13:13
@tisfeng tisfeng merged commit 11fd868 into dev Mar 30, 2024
5 checks passed
@tisfeng tisfeng deleted the revert-418-feautre_tips_view branch March 30, 2024 14:05
AkaShark added a commit to AkaShark/Easydict that referenced this pull request Apr 7, 2024
tisfeng added a commit that referenced this pull request May 19, 2024
* Revert "Revert "Feautre tips view (#418)" (#477)"

This reverts commit 11fd868.

* fix: #452 blank for tips view

* feat: add type of tips cell

* feat: support tips cell update cell type

* fix: when  selection translation the text is empty set special tips cell type

* fix: update show tips status

* fix: resolve merge dev

* fix: update action type

* pref: stash ocr error tips

* fix: control show tips code

* fix: update view

* fix: add mini cell height

* fix: tips cell type

* fix: remove unuse code

* fix: called showTipsView when showTips == true

* fix: display when user just want show query window again

* perf: improve displaying tips view

* perf: do not reload tableView when starting query text

* perf: remove unused Localizable.xcstrings

* perf: close tips view when just updating query text

* perf: close tips view when starting OCR image

* chore(deps): bump rexml in the bundler group across 1 directory

Bumps the bundler group with 1 update in the / directory: [rexml](https://github.com/ruby/rexml).


Updates `rexml` from 3.2.6 to 3.2.8
- [Release notes](https://github.com/ruby/rexml/releases)
- [Changelog](https://github.com/ruby/rexml/blob/master/NEWS.md)
- [Commits](ruby/rexml@v3.2.6...v3.2.8)

---
updated-dependencies:
- dependency-name: rexml
  dependency-type: indirect
  dependency-group: bundler
...

Signed-off-by: dependabot[bot] <support@github.com>

* perf(UI): add icon for disableTipsView toggle

* perf(UI): adjust advanced tab height

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Tisfeng <tisfeng@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jerry <89069957+Jerry23011@users.noreply.github.com>
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.

None yet

3 participants