- [time picker](http://doc.eux.baidu.com/app/list/174e560e528e691b8a/w20dr559_7bxkv6)
- [icode 分支](https://console.cloud.baidu-int.com/devops/icode/repos/baidu/be-fe/brick/merge/time-picker%2Fver-2021...develop)

# 01-29

## input icon 颜色

- (x) close
  - time icon 不改变
  - close input => 加入 className : .icon-input-close
    - var: form-control-clear-icon-button-color
    - 参考 _base
  - 追加两个 vars 用于控制 close x 的颜色
- (-) 是否能统一修改 input suffix?

## md size 支持

- (x) test
  - (x) snapshot 修复
  - (x) 补充 size 的转化逻辑
- (x) item 变高了
  - 缺 md 高度
  - (x) 参考 select 之类的实现
    - [select](/components/select/src/style.scss) #100
- scss 的处理 ( size 循环 )
  - (x) 需要抽离出一些跟随 size 的 scss 变量
  - (x) context, theme ?
- (x) font
- (x) UI
  - s :
    - height : 292 = 28 * 10 + 12
    - item (s) : 64 * 28
    - font : 12
  - m :
    - height : 332 = 32 * 10 + 12
    - item (m) : 64 * 32
    - font : 14
- (x) 切图工具的热键帮助, 一定得有 
  - 连我自己都忘了
  - 使用 popper
- (x) 提取 组件 关键 scss 变量 => brick style 中
- (x) class : sm, md
  - (x) popup class
- (x) sass : sm, md 生成
- (x) demo 修复
- (x) 需要保证非 sm, md 时, 适配一下
  - xs => sm
  - lg, xl => md
  - 其他 => md
  - 此处应体现在逻辑层, 不应该在 sass 里生成所有样式
  - () @UI: 需要考虑 input 的 xs, lg, xl ??
    - @note: input 框, 要跟 input 走
  - (x) 先实现

## 颜色梳理

- 悬浮 : #f5f5f5 => which ?
- 点击 : #eee => which ?
  - 色值 => color (function?) => base palette.scss
  - 对应关系
  - 有 2~4 个 mapping : 直接语义 => 间接语义 => 全局语义 => 色板
- 已选前景 : #4C84FF => which ?
- time-picker-aside-item-text-hover
  - {comp}-{area}-{element}-{status}
  - time-picker-aside-item-text-hover
  - (@) time-picker-aside-item-text-active
    - (x) 如何实现, :active
    - (x) 为什么有两个值???? : 错误引入, 去除
      - 追溯 : blame => next => blame
      - ![img_1.png](image/0210-ui-check/img_git-blame.png)
  - time-picker-aside-item-text-selected
    - text, bg
- 位置
  - packages/style/src/themes/_time-picker.scss

## all

- time picker
  - () 部分问题 q @UI: ?
  - () UI 问题梳理
    - input 相关
      - (x) input 样式, 应是对齐现有的样式了
      - (x) icon UI 稿不符?
        - ![img.png](image/0210-ui-check/img.png)
      - (x) 是否加 placeholder? (input, 应该不用操心)
        - (x) 默认 placeholder
        - (x) 确保 doc 说明显示
          - 好像没出来??
      - (x) icon 色值不一致? (input) 后续其他话题继续
        - ![img_2.png](image/0210-ui-check/img_2.png)
      - (x) 已选焦点态 icon : UI 有问题
        - ![img_3.png](image/0210-ui-check/img_3.png)
    - popper 区域
      - (!) 高亮颜色不一样
        - ![img_4.png](image/0210-ui-check/img_4.png)
        - (!) @UI: 确定具体色值
        - (x) item 背景 / 前景色
          - ![img_6.png](image/0210-ui-check/img_6.png)
          - (x) scss 变量的逻辑处理
            - 去除 "无用变量", 增加 active 背景态
        - (x) 已选点击缺失, 已选态 => 去背景
          - ![img_7.png](image/0210-ui-check/img_7.png)
      - (x) size : 变窄了?
        - ![img_5.png](image/0210-ui-check/img_5.png)
        - 不是问题 : md 下也统一为 192, 非 200了
        - 并且, 原本也没有 md, 所以不算是问题
      - (!) 无已选焦点态 @UI:
      - (x) 阴影应该跟 popper 走, 无误
      - (x) ui bug, 下 padding
        - ![img.png](image/0210-ui-check/img_footer.png)
        - ![img.png](image/0210-ui-check/img_footer-2.png)
        - wtf, 这个有什么构成的?
        - font 造成的问题?
          - 最小复现圈 
            - ![img.png](image/0210-ui-check/img_reproduce-recipe.png)
        - body: 22px line-height
        - 32px, 会吧 padding 顶上去?
        - 加入 vertical-align: middle
      - (x) demo 中 p : margin-bottom 需要补回一个 1em 给它
    - (-) 可能的 文字偏移? (先这样)
    - (x) 直觉上的合理性
    - (x) demo 调整需求
      - (x) size : md, sm
    - (x) 实现逻辑
- @doc: really important!!!
  - IDEA 中 md 文件名, 如果是 "中文" 会重构时的问题
  - () 是否反馈?
- @doc: IDEA, keymap
  - markdown
    - preview + editor : atl + 2
    - editor : alt + 1
    - scroll sync : alt + `
    - 触发执行效率 : key > UI 操作
    - 动态的定位效率 : UI 操作 > key
- () 分为⼩号（S）、中号（M）2种规格，其中以12号文字为基准的界面推荐使用小号为默认尺寸，以14号文字为基准的界面推荐使用中号为默认尺寸。
  - 这个目前的处理逻辑是什么?
- @doc: 对比 (找 soft wrap)
  - hotkey : 记忆实际的操作
  - GUI : 很多的 menu, where?
  - 记得 soft wrap keyword => 匹配
- @note: QA 信息撮合方式?
- @note: 感悟
  - token 很重要, 但是更多的是对于 U, 即 "本身 U 设计, 也应该是系统化, 有所关联和逻辑, 有所语义的"
  - 但如果希望, U 与 FE mapping 一套 token, 那是不现实的?
    - 工作之一, 就是在做 mapping 的关系
  - 内部问题 > 映射问题
  
# 切图模块

- () 引入切图模块
  - (x) base 图 + 红框
  - (x) reset 时, 也恢复混合模式到初始状态
  - (x) 允许图片左边距超出边界
  - (x) 也需要挂载到 body
    - (x) 看 portal
      - [portal](https://reactjs.org/docs/portals.html)
    - (x) 挂载到 body 的合理性 
      - [不太合理](https://stackoverflow.com/questions/49504546/is-it-safe-to-use-reactdom-createportal-with-document-body)
      - [use portal](https://www.jayfreestone.com/writing/react-portals-with-hooks/)
      - () @吴俊: 但是好像现在就是这么用的?
  - () 默认选择第一个图片? 
    - 或者 state 里的 currImage

# possible

- () 如何, 明确, ignore?