Skip to content

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

Merged
3 commits merged into
xpressengine:developfrom
SoneYours:mobilesetting
Apr 14, 2015
Merged

모바일뷰를 사용하지 않는 페이지에서 모바일최적화 버튼이 노출되는 문제점 개선.#1169
3 commits merged into
xpressengine:developfrom
SoneYours:mobilesetting

Conversation

@bjrambo

@bjrambo bjrambo commented Jan 10, 2015

Copy link
Copy Markdown
Contributor

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

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

모바일 뷰어 사용중에만 모바일에 최적화된 화면 버튼을 실행 하도록 개선.
Comment thread classes/module/ModuleHandler.class.php Outdated

Copy link
Copy Markdown

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

bjrambo commented Jan 13, 2015

Copy link
Copy Markdown
Contributor Author

코드 설명,

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.

1 participant