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

모바일뷰를 사용하지 않는 페이지에서 모바일최적화 버튼이 노출되는 문제점 개선. #1169

Merged
3 commits merged into from Apr 14, 2015

Conversation

bjrambo
Copy link
Contributor

@bjrambo bjrambo commented Jan 10, 2015

db_info 옵션을 통해서 모바일뷰를 사용하지 않는 페이지에서 기존의 모듈 설정에 의해 모바일 최적화버튼이 노출되는 문제점이 있습니다.

이를 개선하여 DB옵션도 같이 불러들여 최적화 화면 버튼을 노출시키지 않도록 했어요.

모바일 뷰어 사용중에만 모바일에 최적화된 화면 버튼을 실행 하도록 개선.
@@ -667,7 +667,8 @@ function procModule()
'dispEditorConfigPreview' => 1,
'dispLayoutPreviewWithModule' => 1
);
if($type == "view" && $this->module_info->use_mobile == "Y" && Mobile::isMobileCheckByAgent() && !isset($skipAct[Context::get('act')]))
$db_info = Context::getDBInfo();
if($type == "view" && $this->module_info->use_mobile == "Y" && Mobile::isMobileCheckByAgent() && !isset($skipAct[Context::get('act')]) && $db_info->use_mobile_view == "Y")
Copy link

Choose a reason for hiding this comment

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

/classes/mobile/Mobile.class.php에 mobile 모드 활성화 상태를 true/false로 반환하는 메소드를 추가해두고 활용하면 좋을 것 같네요.

@ghost ghost added the type/enhancement label Jan 13, 2015
@ghost ghost added this to the 1.7.10 milestone Jan 13, 2015
@ghost ghost self-assigned this Jan 13, 2015
@bjrambo
Copy link
Contributor Author

bjrambo commented Jan 13, 2015

코드 설명,

classes/mobile/Mobile.class.php
재일 아래 모바일옵션 사용을 위한 메소드 추가와 동시에
위의 DB옵션을 불러오는것을 메소드불러오도록 했음
self::isMobileEnabled()
을 사용할려다 정확한 이름 정식적으로 불러오도록 수정.

classes/module/ModuleHandler.class.php
역시 메소드에서 $db_use_mobile 값이 true 일경우에만 문구 뜰 수 있도록 수정

말씀주신 modules/admin.admin.view.php
의 경우 수정을 하지 않은 이유는 이 부분은 DB_info의 옵션을 불러들여 이 항목들을 보여주는 구간으로 판단되어서 수정이후 true, False 수정 옵션까지 볼필요 없다고 판단되었습니다.

그 이외 modules/menu/menu.admin.controller.php
에서 DB_info -> 모바일 옵션메소드 사용하도록 했음.

전체 버그 확인.

모바일접속 -> 주소끝 ?m=0 으로 pc화면 출력된 상태에서 아래 옵션 유무에 따른 동작 확인

PC버전에서 관리자페이지 -> menu.admin.controller.php의 수정사항인 메뉴 인설트 부분에 따른 모바일 뷰 사용의 채크 유무 확인, 이상없음

모바일에서 접속시 옵션에 상관없이 모바일화면을 볼 수 잇는 문제점이 발생되지 않았는가? -> 정상적으로 작동하고, 위와 같은 문제점 발견 X

@ghost ghost modified the milestones: 1.8.0.beta.1, 1.7.10 Jan 30, 2015
@ghost ghost modified the milestones: 1.8.0.beta.1, 1.8.0.beta.2 Mar 3, 2015
@ghost ghost modified the milestones: 1.8.0.beta.2, 1.8.0-beta.3 Mar 13, 2015
@ghost ghost modified the milestones: 1.8.0-beta.3, 1.8.1, 1.8.0 Mar 23, 2015
@ghost ghost modified the milestones: 1.8.0, 1.8.1 Apr 8, 2015
ghost pushed a commit that referenced this pull request Apr 14, 2015
모바일뷰를 사용하지 않는 페이지에서 모바일최적화 버튼이 노출되는 문제점 개선.
@ghost ghost merged commit d1a1dd0 into xpressengine:develop Apr 14, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant