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

添加移动端 #95

Merged
merged 4 commits into from
Jul 29, 2015
Merged

添加移动端 #95

merged 4 commits into from
Jul 29, 2015

Conversation

Yannyezixin
Copy link
Member 

  • 添加移动端(非响应式)

@Yannyezixin
Copy link
Member Author

@bcho ~~~

bcho
bcho reviewed Jul 28, 2015
View reviewed changes
backend/app/controllers/FrontBaseController.php
public function isMobile()
{
// 如果有HTTP_X_WAP_PROFILE则一定是移动设备
if (isset ($_SERVER['HTTP_X_WAP_PROFILE'])) {
Copy link
Member

Choose a reason for hiding this comment

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

不是 wap 吧？手机 wifi 下很多都是其他类型的了…… 你试试判断 UA

Copy link
Member Author

Choose a reason for hiding this comment

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

这个也有可能获取不到，下面有判断UA

Copy link
Member

Choose a reason for hiding this comment

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

但我没看到有判断微信的 UA 啊…… 你这个比较老了吧很多机型都不用考虑的了

Copy link
Member Author

Choose a reason for hiding this comment

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

虽然没有，但是这个微信是可以访问的。OK的

Copy link
Member

Choose a reason for hiding this comment

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

🆗

bcho added a commit that referenced this pull request Jul 29, 2015
@bcho
Merge pull request #95 from Yannyezixin/feature/moblie-view
0d2d6dd 
添加移动端
@bcho bcho merged commit 0d2d6dd into vtmer:develop Jul 29, 2015
@Yannyezixin
Copy link
Member Author

@bcho 那个测试地址是哪来着了？

@bcho
Copy link
Member

bcho commented Jul 29, 2015

@Yannyezixin 不记得了哈哈哈哈你在工作室那个 vps 上搭一个吧

@Yannyezixin
Copy link
Member Author

@bcho 那个不是用了你的youknowmymind.com的子域名吗？看下记录..?

@bcho
Copy link
Member

bcho commented Jul 29, 2015

freshman-dev.youknowmymind.com

2015-07-29 18:03 GMT+08:00 Yann [email protected]:

@bcho https://github.com/bcho 那个不是用了你的youknowmymind.com的子域名吗？看下记录..?

—
Reply to this email directly or view it on GitHub
#95 (comment).

@Yannyezixin
Copy link
Member Author

@bcho 还没有更新过去？

@bcho
Copy link
Member

bcho commented Jul 29, 2015

不知道…… 应该是吧，你自己去搭建一个吧 没空弄现在

2015-07-29 18:11 GMT+08:00 Yann [email protected]:

@bcho https://github.com/bcho 还没有更新过去？

—
Reply to this email directly or view it on GitHub
#95 (comment).

@Yannyezixin
Copy link
Member Author

@bcho 你更新过去吧亲~

@bcho
Copy link
Member

bcho commented Jul 30, 2015

没时间……

2015-07-30 14:04 GMT+08:00 Yann [email protected]:

@bcho https://github.com/bcho 你更新过去吧亲~

—
Reply to this email directly or view it on GitHub
#95 (comment).

@Yannyezixin
Copy link
Member Author

那你把数据导出来给我咯

@bcho
Copy link
Member

bcho commented Jul 30, 2015

OK, 我今晚下班了发给你

2015-07-30 14:26 GMT+08:00 Yann [email protected]:

那你把数据导出来给我咯

—
Reply to this email directly or view it on GitHub
#95 (comment).

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Yannyezixin @bcho