PHPfox-Official / mobile-app-issues

phpFox Mobile App bug tracker
0 stars 0 forks source link

Tables removed from Blog in iOS app #3

Closed jwpwhite closed 3 years ago

jwpwhite commented 4 years ago

What's happened?

Tables are removed from Blog in iOS app

Steps to reproduce:

  1. Create a blog with a table on the website
  2. Open the same blog in the iOS app

What's expected?

I would not expect blogs to be removed as they are so widely used

Browsers and Devices tested

iOS app version 1.5

Server information

AWS Linux, php7.2

phpFox version

phpFox 4.7.7

harrison05 commented 4 years ago

Hi @jwpwhite ,

Currently, we don't support display table on Mobile App. This is a list tags won't work on item's description on App:

['head', 'scripts', 'audio', 'video', 'track', 'embed', 'object', 'param', 'source', 'canvas', 'noscript', 'caption', 'col', 'colgroup', 'table', 'tbody', 'td', 'tfoot', 'th', 'thead', 'tr', 'button', 'datalist', 'fieldset', 'form', 'input', 'label', 'legend', 'meter', 'optgroup', 'option', 'output', 'progress', 'select', 'textarea', 'details', 'diaglog', 'menu', 'menuitem', 'summary']

Regards.

jwpwhite commented 4 years ago

Hi @harrison05 Can you provide some detail as to why these have been disabled and if there is a plan to support them in the future? Cheers John

spirogg commented 3 years ago

@harrison05 @jwpwhite any update to : Can you provide some detail as to why these have been disabled and if there is a plan to support them in the future?

spirogg commented 3 years ago

@harrison05 I forgot @jwpwhite added this and I added it here https://github.com/PHPfox-Official/phpfox-v4-issues/issues/2964

Can you tell us when your going to get this fixed otherwise don't include blogs in IOS and Android app.

please do let us know what's the eta time for a solution

you can't say its not possible because every other website app that has a blog on there desktop website, works very well.

regards, Spiro

tonyphpfox commented 3 years ago

Hi @spirogg @jwpwhite

First, the reason why the mobile app disables these tags because these are complex HTML tags that we can not truly match 100% into native (Android/iOS) View. So for the first time, we encounter these, we simply ignore/strip these tags on mobile.

But, we think it would be better if we enabled these tags, especially tables tags in the mobile app, so we are considering a new solution for these matching. We'll implement this improvement on the mobile app as soon as possible as we can.

Regards.

spirogg commented 3 years ago

@tonyphpfox thank you. We really need this to work. I appreciate your feedback on this issue

Regards,

Spiro

jwpwhite commented 3 years ago

Thanks @tonyphpfox, looking forward to the update :) Kind regards John