Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

ngWig with angular-material #8593

Closed
kpgarrod opened this issue May 30, 2016 · 5 comments
Closed

ngWig with angular-material #8593

kpgarrod opened this issue May 30, 2016 · 5 comments

Comments

@kpgarrod
Copy link

kpgarrod commented May 30, 2016

I would like to be able to use a simple wysiwig editor in my a-m project. ngWig is very lightweight, simple to implement in place of textareas and fits the requirement perfectly except for a couple of 'minor' issues:

Actual behavior:

  • What is the issue? *
    This is a copy of the issue I posted in the ngWig repo here
  • When I include ng-wig inside a form (with or without md-input-container) the editor does not become focused when tabbing through the form.
  1. On a mobile device the keyboard does not appear when focusing on the editor.
  • What is the expected behavior?
    ngWig in a project containing angular-material should behave as it does without angular-material. Ideally ideally it should be possible to contain an ng-wig editor within an md-input-container

CodePen or Steps to reproduce the issue: *

Angular Versions: *

  • Angular Version:
    1.5.x
  • Angular Material Version:
    1.1.0-rc4

Additional Information:

  • Browser Type: *
    android /chrome on android
  • Browser Version: *
  • OS: *
    android
  • Stack Traces:

Shortcut to create a new CodePen Demo.
Note: * indicates required information. Without this information, your issue may be auto-closed.

Do not modify the titles or questions. Simply add your responses to the ends of the questions.
Add more lines if needed.

@devversion
Copy link
Member

Closing this as a duplicate of #8487

@stevermeister
Copy link

@devversion it's not a duplicate, it's exact issue for angular-material.

contenteditable field does not get the focus on mobile devices when you click on it.

(But it works if you just remove angular-material module from your dependancies)

@myknbani
Copy link

It is not a dupe of #8487 but rather of issue #5937. There's also this PR #5939

@stevermeister
Copy link

@myknbani thank you for the link!

@devversion
Copy link
Member

@stevermeister Thanks for the clarification. The issue was not very clear at beginning (before its editing). It sounded more like a feature request, to include a RTE in the core.

As @myknbani shared the link of a previous and more generic issue - I'll reopen #5937 (which was closed within our Spring Cleaning)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants