search

sugarlabs / infoslicer

GNU General Public License v2.0
2 stars 10 forks source link

Bug: doesn't parse tables properly #12

Closed rhl-bthr closed 5 years ago

rhl-bthr commented 6 years ago

Steps to reproduce:

  1. Open the activtiy and search for something in the Get Article From: English Wikipedia Example: juhu
  2. The result looks something like - sugar-inforclicer

Issue: The table, tr and td tags are not handled properly

Reproduced on:

Possible Solutions:

  1. Design a format to present tables neatly. The amount of text in all cells needs to be taken care, for proper indentation and size of columns.
  2. Ignore tables entirely, as the search says Get Article From and this table is mostly a summary of whats given below. This is not preferred as there might be relevant data in other tables.
nswarup14 commented 5 years ago

@pro-panda I did look into the bug that you have described and I believe that the second one, amongst the possible solution that you have listed is actually being implemented. I managed to take a screen shot as well.

Tested on Ubuntu 18.04, Sugar v0.112 using rDesktop screenshot from 2019-03-06 15-27-12

rhl-bthr commented 5 years ago

Thanks, tested again. I agree