PocketSword
  1. PocketSword
  2. PS-41

images do not display in all modules

    Details

    • Type: Bug Bug
    • Status: Closed (View Workflow)
    • Priority: Minor Minor
    • Resolution: Invalid
    • Affects Version/s: 1.2.2b3
    • Fix Version/s: None
    • Component/s: Module Display
    • Labels:
      None

      Description

      PS displays module images in some contexts but not all. In my NET and NETnote modules, NET has an image in the heading of all 1:1, and NETnote has an image in all n:1. The modules are ThML. NET's content has this form (from Gen 1:1):

      <div class="sechead"><center><img src="/images/netbible.jpg" /></center>The Creation of the World</div>

      NETnote's has this form:

      <center><img src="/images/msstrip.gif" /></center>

      Perhaps it is significant that the image in NET is in pre-verse heading between <div></div>. cf. display as seen in e.g. Xiphos.

        Activity

        Hide
        Nic Carter added a comment -

        Hmmm, is this bug due to PocketSword not displaying headings?
        Also, if you can tweak your module a tiny bit, does it still occur if it's encoded as:
        <div class="sechead"><center><img src="images/netbible.jpg" /></center>The Creation of the World</div>
        [note the lack of the initial "/" in the src tag]

        Show
        Nic Carter added a comment - Hmmm, is this bug due to PocketSword not displaying headings? Also, if you can tweak your module a tiny bit, does it still occur if it's encoded as: <div class="sechead"><center><img src="images/netbible.jpg" /></center>The Creation of the World</div> [note the lack of the initial "/" in the src tag]
        Hide
        karl kleinpaste added a comment -

        Oh...duh. PS doesn't display headings.

        OK, so that's why. So maybe this bug needs to be changed: Should offer to display headings, with an option just like displaying Strong's/morph.

        Removing the initial / in the image file pathname causes grief in the engine because it fails to generate the right filename in the filtered result. This isn't considered a bug; it's that module content is considered to have a relative root.

        Show
        karl kleinpaste added a comment - Oh...duh. PS doesn't display headings. OK, so that's why. So maybe this bug needs to be changed: Should offer to display headings, with an option just like displaying Strong's/morph. Removing the initial / in the image file pathname causes grief in the engine because it fails to generate the right filename in the filtered result. This isn't considered a bug; it's that module content is considered to have a relative root.
        Hide
        Nic Carter added a comment -

        ok, so you do need the initial "/". But, are these two modules simply not working cause headers aren't displayed? If that's true, I'll close this issue. You can add a new issue if you want headers to appear (I do want them there and am hoping to get onto it for v1.3!).

        Show
        Nic Carter added a comment - ok, so you do need the initial "/". But, are these two modules simply not working cause headers aren't displayed? If that's true, I'll close this issue. You can add a new issue if you want headers to appear (I do want them there and am hoping to get onto it for v1.3!).
        Hide
        karl kleinpaste added a comment -

        If you're tracking a separate issue for introducing heading display, then just close this one. It seems likely that my NET will display images normally once you provide headings.

        If it doesn't, then I'll file another bug...

        Show
        karl kleinpaste added a comment - If you're tracking a separate issue for introducing heading display, then just close this one. It seems likely that my NET will display images normally once you provide headings. If it doesn't, then I'll file another bug...
        Hide
        Nic Carter added a comment -

        This is a headings issue, not an image display issue. still an issue, tho...

        Show
        Nic Carter added a comment - This is a headings issue, not an image display issue. still an issue, tho...

          People

          • Assignee:
            Nic Carter
            Reporter:
            karl kleinpaste
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: