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

feat: add color by offsettime,emoji by outsource or training,job item sort desc by time #9

Merged

Conversation

lastsunday
Copy link
Contributor

CHANGELOG:

  1. 使用emoji進行公司標識:太陽為普通公司,陰天為外包或培訓機構

  2. 根據距離更新時間天數改變背景顔色:<=7:yellowgreen,<=14:green,<=28:orange,<=56:red,其他:gray

  3. 適配智聯招聘

  4. 適配前程無憂

  5. 修復時間顯示邏輯

  6. 根據時間降序顯示

顯示效果如下:

  • BOOS直聘

boss-show-time-merge-request-001

  • 智聯招聘
    boss-show-time-merge-request-003

  • 前程無憂

boss-show-time-merge-request-002

@tangzhiyao
Copy link
Owner

tangzhiyao commented May 7, 2024

image
https://we.51job.com/pc/search?jobArea=080200&keyword=&searchType=2&sortType=0&metro= 前程无忧这里有个问题,出现两个标签了

@tangzhiyao
Copy link
Owner

请教一个问题,我看你这排序用的 order 属性,是一开始就用的,还是说直接操作dom进行排序有问题才改用的?

@lastsunday
Copy link
Contributor Author

image https://we.51job.com/pc/search?jobArea=080200&keyword=&searchType=2&sortType=0&metro= 前程无忧这里有个问题,出现两个标签了

这个问题我这边环境复现不了。

@lastsunday
Copy link
Contributor Author

请教一个问题,我看你这排序用的 order 属性,是一开始就用的,还是说直接操作dom进行排序有问题才改用的?

一开始做这个排序需求的时候就用这个属性,想到操作dom就觉得麻烦,侵入性太强,所以pass了。所以才采用flexbox order这种实现方式。

@tangzhiyao
Copy link
Owner

好的,我这先合了。问题我也没复现了,我记录下后续关注着

@tangzhiyao tangzhiyao merged commit 99bb6da into tangzhiyao:main May 7, 2024
@lastsunday lastsunday deleted the feature_sort_by_lastmodifytime branch May 29, 2024 15:17
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.

2 participants