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.

Sign up for GitHub

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

Jump to bottom

修复 scrollToHash 当锚点的祖先存在定位元素时不能滚动到锚点的问题 #675

Merged
merged 4 commits into from
Aug 13, 2019
Merged

修复 scrollToHash 当锚点的祖先存在定位元素时不能滚动到锚点的问题 #675

merged 4 commits into from
Aug 13, 2019

Conversation

Brunoon
Copy link
Contributor

@Brunoon Brunoon commented Aug 7, 2019

相关 ISSUE: （ISSUE 链接地址）
#673

1、升级点 （清晰准确的描述升级的功能点）

使用对 scrollTo 封装的 handleScrollTo，该方法通过 util.rect.getElementRect 获取元素到页面顶部的距离，而不是使用 offsetTop，能够正确滚动到元素所在位置。

2、影响范围 （描述该需求上线会影响什么功能）

使用 scrollToHash 方法进行滚动的锚点。

3、自测 Checklist

4、需要覆盖的场景和 Case

  • 是否覆盖了 sf 打开 MIP 页
  • 是否验证了极速服务 MIP 页面效果

5、自测机型和浏览器

  • 是否覆盖了 iOS 系统手机
  • 是否覆盖了 Android 系统手机
  • 是否覆盖了 iPhone 版式浏览器（比如 QQ、UC、Chrome、Safari、安卓自带浏览器）
  • 是否覆盖了手百

@Brunoon Brunoon changed the base branch from master to dev August 7, 2019 05:32
@clark-t
Copy link
Contributor

clark-t commented Aug 8, 2019

增加个 testcase 覆盖以下

@mipengine mipengine deleted a comment from coveralls Aug 8, 2019
@mipengine mipengine deleted a comment from mipengine-bot Aug 8, 2019
@mipengine mipengine deleted a comment from mipengine-bot Aug 8, 2019
@mipengine mipengine deleted a comment from coveralls Aug 8, 2019
@Brunoon Brunoon merged commit d1f2439 into dev Aug 13, 2019
Brunoon added a commit that referenced this pull request Aug 13, 2019
@Brunoon
8.13 上线 (#678)
bcb4d3a 
* 修复 scrollToHash 当锚点的祖先存在�定位元素时不能滚动到锚点的问题 (#675)
clark-t added a commit that referenced this pull request Aug 27, 2019
@clark-t
8.27 核心上线 (#690)
6e98d19 
* 单页跳转透传 hash

* add cambrian logo and title

* add cambrian logo and title

* fixed: 修复 iOS 百度 APP 下的页面无法滚动的问题

* [feat] MIP 交互机制升级 (#656)

* add parser

* 添加全局行为

* 增加 event 语法规则

* 添加 navigateTo，fix code

* rewrite mip event

* finish mip event parse

* add visitor

* update

* add arrow

* fix property & callexpression bug

* 整合全局方法

* fix callexpression bug

* 调整代码

* update code

* fix objectexpression bug

* update whitelist

* update test case

* add test case

* modify test case

* add DOM

* 修复 Identifier bug

* 修复 Identifier bug

* add mip-action

* update code

* add argument scope

* 修改路径

* update path

* 打通 action 数据传递

* fix mdata bug

* fix element action bug

* 增加兼容逻辑

* 重写 mip-bind 方法

* update code

* add setData parse fallback

* update bindStyle

* update bindStyle

* update bindStyle

* 增加 watch

* 增加 input  的双向绑定

* 删除无用文件

* 修改 mip-bin 入口

* update code

* 拆分 bind.js

* 拆分 mip-bind/bind.js

* add input global event

* update event action spec

* 修复 style binding

* remove only

* update input global event and toggleVisibility

* 去除 mip-shell/render.js 调用 MIP. 的逻辑

* change lexer from class to function

* add test page

* 优化 lexer 机制

* 修改 lexer 策略

* update test page

* update unit test

* 补 testcase

* 补全 testcase

* 修改 ArrayLiteral 对最后一个 comma 的处理

* 增加测试描述

* add unit test

* add complex function testcase

* fix scrollTO hidden element

* fix can not find mip1 custom element action

* 修复 setData 接收 Vue 响应式对象时疯狂报错的问题

* fix isObject

* 修复 mip-bind 覆盖 MIP.setData 的bug

* 修复  lgtm 所提示的循环中使用 splice 的问题

* fix lgtm alerts

* 修复 bind & vnavigateTo 在 IOS8 下面无法正常工作的兼容性 bug

*  移除 mip-img popup 对 loaded 的筛选 (#668)

* 移除 mip-img popup 对 loaded 的筛选

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* 修复 scrollToHash 当锚点的祖先存在�定位元素时不能滚动到锚点的问题 (#675)

* [feat] mip-bind 功能升级 (#685)

* add scoped mip-data

* update mip-bind dom diff

* 增加 mip-data 的 refresh

* update dom watch

* update dom watcher

* change scoped to scope

* 修复部分 test case

* 优化报错信息

* fix docs

* 增加单测覆盖率

* 补充 testcase

* 新增 mip-data fetchjsonp 支持

* 去掉 mip-data 的 JSONP 支持
clark-t added a commit that referenced this pull request Oct 8, 2019
@clark-t
MIP 核心 10.08 上线 (#705)
3ef6a9b 
* 单页跳转透传 hash

* add cambrian logo and title

* add cambrian logo and title

* fixed: 修复 iOS 百度 APP 下的页面无法滚动的问题

* [feat] MIP 交互机制升级 (#656)

* add parser

* 添加全局行为

* 增加 event 语法规则

* 添加 navigateTo，fix code

* rewrite mip event

* finish mip event parse

* add visitor

* update

* add arrow

* fix property & callexpression bug

* 整合全局方法

* fix callexpression bug

* 调整代码

* update code

* fix objectexpression bug

* update whitelist

* update test case

* add test case

* modify test case

* add DOM

* 修复 Identifier bug

* 修复 Identifier bug

* add mip-action

* update code

* add argument scope

* 修改路径

* update path

* 打通 action 数据传递

* fix mdata bug

* fix element action bug

* 增加兼容逻辑

* 重写 mip-bind 方法

* update code

* add setData parse fallback

* update bindStyle

* update bindStyle

* update bindStyle

* 增加 watch

* 增加 input  的双向绑定

* 删除无用文件

* 修改 mip-bin 入口

* update code

* 拆分 bind.js

* 拆分 mip-bind/bind.js

* add input global event

* update event action spec

* 修复 style binding

* remove only

* update input global event and toggleVisibility

* 去除 mip-shell/render.js 调用 MIP. 的逻辑

* change lexer from class to function

* add test page

* 优化 lexer 机制

* 修改 lexer 策略

* update test page

* update unit test

* 补 testcase

* 补全 testcase

* 修改 ArrayLiteral 对最后一个 comma 的处理

* 增加测试描述

* add unit test

* add complex function testcase

* fix scrollTO hidden element

* fix can not find mip1 custom element action

* 修复 setData 接收 Vue 响应式对象时疯狂报错的问题

* fix isObject

* 修复 mip-bind 覆盖 MIP.setData 的bug

* 修复  lgtm 所提示的循环中使用 splice 的问题

* fix lgtm alerts

* 修复 bind & vnavigateTo 在 IOS8 下面无法正常工作的兼容性 bug

*  移除 mip-img popup 对 loaded 的筛选 (#668)

* 移除 mip-img popup 对 loaded 的筛选

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* 修复 scrollToHash 当锚点的祖先存在�定位元素时不能滚动到锚点的问题 (#675)

* [feat] mip-bind 功能升级 (#685)

* add scoped mip-data

* update mip-bind dom diff

* 增加 mip-data 的 refresh

* update dom watch

* update dom watcher

* change scoped to scope

* 修复部分 test case

* 优化报错信息

* fix docs

* 增加单测覆盖率

* 补充 testcase

* 新增 mip-data fetchjsonp 支持

* 去掉 mip-data 的 JSONP 支持

* 统一 MIP 表达式的语法形式 (#701)

* 增强 MIP 表达式语法

* 调整文章

* 增加 String.prototype.trim 在 MIP 表达式中的支持
clark-t added a commit that referenced this pull request Jan 7, 2020
@clark-t @mip-workflow
@zoumiaojiang @Brunoon
2020.01.07 MIP 核心上线修复 MIP Shell 头部污染问题 (#735)
2aa025d 
* 单页跳转透传 hash

* add cambrian logo and title

* add cambrian logo and title

* fixed: 修复 iOS 百度 APP 下的页面无法滚动的问题

* [feat] MIP 交互机制升级 (#656)

* add parser

* 添加全局行为

* 增加 event 语法规则

* 添加 navigateTo，fix code

* rewrite mip event

* finish mip event parse

* add visitor

* update

* add arrow

* fix property & callexpression bug

* 整合全局方法

* fix callexpression bug

* 调整代码

* update code

* fix objectexpression bug

* update whitelist

* update test case

* add test case

* modify test case

* add DOM

* 修复 Identifier bug

* 修复 Identifier bug

* add mip-action

* update code

* add argument scope

* 修改路径

* update path

* 打通 action 数据传递

* fix mdata bug

* fix element action bug

* 增加兼容逻辑

* 重写 mip-bind 方法

* update code

* add setData parse fallback

* update bindStyle

* update bindStyle

* update bindStyle

* 增加 watch

* 增加 input  的双向绑定

* 删除无用文件

* 修改 mip-bin 入口

* update code

* 拆分 bind.js

* 拆分 mip-bind/bind.js

* add input global event

* update event action spec

* 修复 style binding

* remove only

* update input global event and toggleVisibility

* 去除 mip-shell/render.js 调用 MIP. 的逻辑

* change lexer from class to function

* add test page

* 优化 lexer 机制

* 修改 lexer 策略

* update test page

* update unit test

* 补 testcase

* 补全 testcase

* 修改 ArrayLiteral 对最后一个 comma 的处理

* 增加测试描述

* add unit test

* add complex function testcase

* fix scrollTO hidden element

* fix can not find mip1 custom element action

* 修复 setData 接收 Vue 响应式对象时疯狂报错的问题

* fix isObject

* 修复 mip-bind 覆盖 MIP.setData 的bug

* 修复  lgtm 所提示的循环中使用 splice 的问题

* fix lgtm alerts

* 修复 bind & vnavigateTo 在 IOS8 下面无法正常工作的兼容性 bug

*  移除 mip-img popup 对 loaded 的筛选 (#668)

* 移除 mip-img popup 对 loaded 的筛选

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* fix: hash 锚点追加的问题&pc 页面空白问题&添加百度 APP  iOS 通信功能

* 修复 scrollToHash 当锚点的祖先存在�定位元素时不能滚动到锚点的问题 (#675)

* [feat] mip-bind 功能升级 (#685)

* add scoped mip-data

* update mip-bind dom diff

* 增加 mip-data 的 refresh

* update dom watch

* update dom watcher

* change scoped to scope

* 修复部分 test case

* 优化报错信息

* fix docs

* 增加单测覆盖率

* 补充 testcase

* 新增 mip-data fetchjsonp 支持

* 去掉 mip-data 的 JSONP 支持

* 统一 MIP 表达式的语法形式 (#701)

* 增强 MIP 表达式语法

* 调整文章

* 增加 String.prototype.trim 在 MIP 表达式中的支持

* 消除 自定义样式 min-width/min-height 对 MIP 固定顶部栏样式所造成的影响 (#732)

Co-authored-by: mip-workflow[bot] <45706949+mip-workflow[bot]@users.noreply.github.com>
Co-authored-by: zoumiaojiang <[email protected]>
Co-authored-by: guozhuorong <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clark-t clark-t clark-t approved these changes

@liujing50 liujing50 liujing50 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Brunoon @clark-t @liujing50