Move "Add comment" and "Details" into modal with "Comments"
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mahara |
Fix Released
|
Wishlist
|
Lisa Seeto |
Bug Description
This is part of the page revamp: https:/
Currently, the "Add comment" and "Details" links at blocks that can have them, takes you to a separate page where you can view the artefact's metadata and can leave comments.
When you view comments, they are already displayed in the modal directly on the page, but you can't leave a comment yet.
The things to be done:
- Combine comments, add comment, Details to "Comments and details" (or just comments or just details depending on what can be displayed) so you only have one link.
- Clicking that link opens up a modal on the page that shows the artefacts details (if there are any) and also the comments, as well as allows you to leave comments if that is enabled. Please see the attached mock-up.
This would also mean to review all the "Details" on blocks as some of them are not needed. For example, clicking the details link next to a plan task only shows that single plan task again without any additional details. The "Details" should only be there when there is additional metadata information but not simply repeating the block content.
The title of the modal (the one in the green bar) would be the blocktype as the title of the block is listed below.
A decision still needs to be made on whether comments can be paginated in the modal or if they are shown all one after another (which might get quite long).
We are not changing anything else from the proposal as that requires Gridstack to be implemented first. This is a standalone usability improvement independent of Gridstack.
Kristina Hoeppner (kris-hoeppner) wrote : | #1 |
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review | #2 |
Changed in mahara: | |
status: | Confirmed → In Progress |
Mahara Bot (dev-mahara) wrote : | #3 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #4 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #5 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #6 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #7 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #8 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #9 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #10 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #11 |
Patch for "master" branch: https:/
Steven (stevens-q) wrote : | #12 |
Commit hash: 013743dffc98b80
Environment tested: Master
Browser tested: Chrome
Theme used: Raw
PRECONDITIONS:
-------
1) Mahara User1 exists and has a portfolio page "1 Column Layout - (Comments testing)" and it is public and able to be copied
2) Portfolio page has the following blocks:
a) Image
b) File(s) to download
c) Folder
d) Journal
e) Journal entry
f) Recent journal entries
g) Tagged journal entries
h) Plans
i) Notes (with tags)
j) My entire resume
k) One resume field
3) User1's resume section is completely filled in with attachments where applicable.
4) Journal exists with Journal entries which have attached files and comments are allowed
5) Site has licenced meta data turned on (admin menu > Configure site > Site options > General)
6) Mahara User3 exists (No Roles)
TEST STEPS: note the idea is that the comments and details are displayed in the flyout modal area
-------
Image block:
1) Log in as Mahara User3
2) Browse to User1' s portfolio page as mentioned above
3) Verify the image block has the following:
- Image displays and is a link that opens the modal ✔
- Link under the image opens the modal: ✔
- [icon] Comments (current number of comments) and details ✔
4) Click the link to make a comment
5) Verify that the modal opens and has the following
- preview image that is a link to download the image ✔
- metadata ✔
- comments (if any) ✔
- Add comment form ✔
6) Make a comment and attach a file to it
7) Verify the comment displays in comment area with download link inside the flyout modal ✔
File(s) to download:
1) Log in as Mahara User3
2) Browse to User1' s portfolio page as mentioned above
3) Verify the files to download block has the following
- Each file title is a link that opens the modal. Hovering over the row gives the option to download the file. ✔
4) Click a file title
5) Verify that the fly out modal has the following:
a) For image files:
- preview image that is a link to download the image ✔
- metadata with download button ✔
- comments (if any) ✔
- Add comment form ✔
b) For other files:
-metadata with download button ✔
6) Verify that attachments can be attached as a comment when there is an add comment form, displays in comment area with download link
Folder:
1) Log in as Mahara User3
2) Browse to User1' s portfolio page as mentioned above
3) Verify that the folder block has the following:
a) The details link no longer appears ✔
b) Each file title is a link that opens the modal. Hovering over the row gives the option to download the file ✔
4) Click the file title of the file you want to make a comment for or view details
5) Verify that the folder block has the following
a) For image files:
- preview image that is a link to download the image ✔
- metadata with download button ✔
- comments (if any) ✔
- Add comment form ✔
b) For other files:
-metadata with download button ✔
6) Verify that attachments can be attached as a comment when there is an add comment form, displays in comment area with download link ✔
NOTE: to view the flyout modal user must click on the title - clicking anywh...
Steven (stevens-q) wrote : | #13 |
BUG #1 User can make a comment without entering text in the comment field
**************
#1 (Chrome)
When a user clicks comment button without entering a comment, an error message is displayed. However it is only for a few seconds before the modal appears to reopen and the error message is hidden. ✘
#2 (Firefox)
When a user clicks comment button without entering a comment, an error message is displayed.
However the error message is displayed on the main page under the modal and a second error message is displayed just under the Add comment (WISIWIG) comment field. This is inconsistent with the rest of the site. ✘
*******
*******
Bug #2 Accessibility issue - User cannot navigate to modal window using keyboard only naviigation
**************
Bug Has been fixed and user can now navigate to the modal and close the modal if they wish. ✔
Lisa Seeto (lisaseeto) wrote : | #14 |
Hi Steven,
I can not reproduce issues #1 or #2 from your last comment (#13). Are there any extra steps I am missing?
Steven (stevens-q) wrote : | #15 |
Mahara Bot (dev-mahara) wrote : | #16 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #17 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #18 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #19 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : | #20 |
Patch for "master" branch: https:/
Rangi Daymond (rangid) wrote : | #21 |
- SS1 Block Comments and details area positioning inconsistent.png Edit (262.4 KiB, image/png)
Patchset: https:/
Browser tested: Firefox
Theme used: Raw
*PRECONDITIONS:
----------------
1) Logged in (non administrative) user1 has a newly created portfolio with:
a) at least two Journals with several Journal entries in each with a couple of shared tags across them and at least one entry with a long title
b) a page that contains the following (populated) blocks:
1.1 Embedded media
1.2 Image
1.3 Image gallery
1.4 PDF
1.5 Some HTML
1.6 Note
1.7 Journal
1.8 Recent journal entries
1.9 Tagged journal entries --> ensure to select the option 'Show journal entries in full' ie set to Yes
2) The portfolio/page is shared with at least one other non administrative user (user2)
*TEST STEPS:
-------------
Step 1) As logged in user1, display the target portfolio/page then select the page 'Details' button (above the 'Edit' button, near top on right)
Step 2) Check the styling, positioning and functionality of the various block comment & detail bars/icons
***ISSUE1:
----------
Expected results: The styling and positioning of the various block comment & detail bars/icons are as follows:
1.1 Embedded media (bar - details only, positioned at top of block) X
1.2 Image (bar - details and comments, positioned at top of block) ✓
1.3 Image gallery (bar - details only, positioned at top of block) X
1.4 PDF (bar - details and comments, positioned at top of block) X
1.5 Some HTML (bar - details only, positioned at top of block) X
1.6 Note (bar - details and comments, positioned at top of block) ✓
1.7 Journal with 1+ Journal entries --> one per entry (icons - details and comments, positioned on top right of entry title) ✓
1.8 Recent journal entries --> one each entry (icons - details and comments, positioned on top right of entry title) ✓
1.9 Tagged journal entries --> one each entry (icons - details and comments, positioned on top right of entry title) X
--> the entry item styling should be as per the Recent journal entry styling X
2.0 Long titles of blocks that use comment and/or detail icons should be readable X
Actual results: please see attached annotated screenshots (filenames prefixed with SS1 & SS2-1, SS2-2 & SS2-3), as expected above except for:
1.1 Embedded media, 1.3 Image Gallery & 1.5 Some HTML: the bar position is incorrect (under instead of above the title when present), see SS1
1.4 PDF should be a bar (not icons) positioned at the top of the block, see SS1
1.9 Tagged journal entries should each have comment (if user selected) and/or detail icons to the right of their (wrapped if necessary) title, see SS2-1, SS2-2 & SS2-3. The entry item styling is not as per the Recent journal entry styling, see SS2-1
2.0 Jounal entries with long titles are partially covered by the comment and/or detail icon/s, see SS2-1.
***ISSUE2:
----------
Step 3) Scroll to view the PDF block and click on the comment icon and add a Comment
Expected results: the comment icon should update to include an incremented comment count value eg from none to 1, 1 to 2 etc. and when the user refres...
Rangi Daymond (rangid) wrote : | #22 |
Rangi Daymond (rangid) wrote : | #23 |
Rangi Daymond (rangid) wrote : | #24 |
Rangi Daymond (rangid) wrote : | #25 |
Rangi Daymond (rangid) wrote : | #26 |
Rangi Daymond (rangid) wrote : | #27 |
- SS5-1 Image Block Comments and details bar displayed before Comment removed.png Edit (266.1 KiB, image/png)
Rangi Daymond (rangid) wrote : | #28 |
- SS5-2 Image Block Comments and details bar displayed with old icon after Comment removed.png Edit (262.2 KiB, image/png)
Rangi Daymond (rangid) wrote : | #29 |
Rangi Daymond (rangid) wrote : | #30 |
- SS1 Exported HTML - image details and comments not accessible.png Edit (231.3 KiB, image/png)
Patchset: https:/
Browser tested: Firefox
Theme used: Raw
TEST SCENARIO: Verify that exported HTML portfolio/pages with image blocks that have Comments and/or Details can be viewed as expected.
--------------
*PRECONDITIONS:
---------------
1) Logged in (non administrative) user1 has a newly created HTML export of a portfolio/page with at least two image blocks (one commented upon - public and private ones - and one not) and one folder block (that contains the image/s used in the image blocks).
2) The portfolio/page is shared with at least one other non administrative user (user2) who has also commented (some private and some public) on the block with prior comments.
3) The portfolio/page is exported as HTML with comments option selected, and the downloaded file unzipped and the index.html file is opened in the browser
**TEST STEPS:
-------------
Verify that the portfolio/page content displays as expected with special focus on the block comments and details.
Step 1. Open the index.html file to view the exported material.
Step 2. Click on the image block's image to view the details and comments.
**Image block HTML export bug: the image block details and comments are not accessible via the image block but they are via the folder block file - please see the attachments (with prefix SS1 shows the console error when the user attempts to click the image in the image block, with prefix SS2 shows the details and comments displayed when user clicks the folder block's image filename).
Rangi Daymond (rangid) wrote : | #31 |
- SS2 Exported HTML - image details and comments ARE accessible via filename.png Edit (254.2 KiB, image/png)
Mahara Bot (dev-mahara) wrote : | #32 |
Patch for "master" branch: https:/
Rangi Daymond (rangid) wrote : | #33 |
- ISS2 Journal block - extra line displayed under block title line.png Edit (162.3 KiB, image/png)
Gerrit/Patchset: https:/
Browser tested: Firefox only at present
Theme used: Raw
*PRECONDITIONS:
----------------
1) Logged in (non administrative) user1 has a recently created portfolio/page with:
a) at least two Journals with several Journal entries in each with a couple of shared tags across them and at least one entry with a long title
b) a page that contains the following (populated) blocks:
1.1 Embedded media
1.2 Image
1.3 Image gallery
1.4 PDF
1.5 Some HTML
1.6 Note
1.7 Journal
1.8 Recent journal entries
1.9 Tagged journal entries --> ensure to select the option 'Show journal entries in full' ie set to Yes
1.10 Long block titles that use comment and/or detail icons should be readable
1.11 File/s to download
1.12 Folder
2) The portfolio/page is shared with at least one other non administrative user (user2)
*TEST STEPS:
-------------
Step A) In each browser, log in as user1 then user2, display the target portfolio/page then select the page 'Details' button (above the 'Edit' button, near top on right)
Step B) Check the styling, positioning and functionality of the various block comment & detail bars/icons
Expected & Actual results: The styling and positioning of the various block comment & detail bars/icons are as follows:
1.1 Embedded media (bar - details and comments, positioned at top of block) click bar for modal ✓
1.2 Image (bar - details and comments, positioned at top of block) click bar or image for modal ✓
1.3 Image gallery (bar - details only, positioned at top of block) click bar for modal or
1.4 PDF (bar - details and comments, positioned at top of block) click bar for modal ✓
1.5 Some HTML (bar - details only, positioned at top of block) click bar for modal ✓
1.6 Note (bar - details and comments, positioned at top of block) click bar for modal ✓
1.7 Journal with 1+ Journal entries --> one per entry (icons - details and comments, positioned on top right of entry title)
click journal entry title or icon for modal - minor problem see Issue 2 below X
1.8 Recent journal entries --> one each entry (icons - details and comments, positioned on top right of entry title)
1.9 Tagged journal entries --> one each entry (icons - details and comments, positioned on top right of entry title)
1.10 Long titles of blocks that use comment and/or detail icons should be readable ✓
1.11 File/s to download --> one each entry (icons - details and comments, positioned on top right of entry title)
1.12 Folder --> sub-folder entry/ies (details icon only, positioned on top right of entry title)
click folder name or details icon for modal displaying folder content - see issue 3 below X
--> filenam...
Rangi Daymond (rangid) wrote : | #34 |
Mahara Bot (dev-mahara) wrote : A change has been merged | #35 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit 953a5eab4671132
Author: Lisa Seeto <email address hidden>
Date: Fri Sep 6 14:25:29 2019 +1200
Bug 1827811: Resume Icons
- get attachment comment counts for bh icons
- display plain text for titles in editing mode
- change the way the commentlink is targetted via js
- add blockid's to resume attachments
- for folder block: add comment count, change titles from
links to plain text when editing
- change notes attachment file titles from links to plain
text when editing
- get rid of the single artefact block title links for html exports
-------
Add fixes:
- blocks that have set types but no artefacts attached don't have modal links
- pdf block is now using block header and comment count is loaded
- block headers are above titles
- tagged journals have icons when in full and matches recent posts styling
- deleting all comments produces correct icon on block header
- html exports should not hang on opening modal
- get rid of warning error re: dock.show() when redirecting from &modal=1 style url
- code review fixes for all previous patches + fix issue with folders block
- added in viewdetailsfilt
- make the details btn mode "sticky" so it stays the save value over multiple pages etc...
- Image Gallery blocks: if displaying a folder of images will get the bar block header and
will have a the fancy box image preview displaying on image click
Change-Id: I425cc5a634e176
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : | #36 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit b82df03d4b572ae
Author: Lisa Seeto <email address hidden>
Date: Tue Sep 3 18:01:12 2019 +1200
Bug 1827811: Embedded Media Task
add the block-header to the block.
remove the download link from page as it now shows in the modal.
add artefactid and blockid to image.tpl
fix issue with data-toggle collapse targeting syntax error.
update active block functionality for block headers
update block-comments-
fix Journals, and tagged entries and recent posts to use icons for block header.
change pdf block to use block header icons
clean up .comments-details class that is no longer needed.
add modal functionality for image gallery blocks for thumbnails and thumbnails square presentation.
add modal functionality for some html block, clear up js for icons that is no longer needed.
add title text to icons and block-header
create activateModalLi
Change-Id: Ib5967d44288557
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : | #38 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit e6b6ecf9186d0f2
Author: Lisa Seeto <email address hidden>
Date: Fri Jul 12 11:32:07 2019 +1200
Bug 1827811: Exports task
This change creates the modals for exported html pages.
Modals are hard-coded onto page with id's of modal_[artefactid]
Bootstrap and jquery files are copied into export js folder.
Profile page is also exported with any modal content.
Add tagged posts modals
get rid of export warning when no $commentcount set in viewposts.tpl
Add recent posts modals so that first journal post was also a modal
Set Parent journal link text to plain text for exports
Add entire resume modals and one resume field modals
Add File(s) to download modals
refactor export_
add metadata but not comments to resume page
add metata (license info) but not comments to journal pages
add comment count default to 0 in .tpl's if comment count not present (during export)
remove updating comments modals for profile page as this is no longer needed.
Change-Id: Iff50138285951a
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : | #39 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit 79201b7d8a132a0
Author: Lisa Seeto <email address hidden>
Date: Mon Jul 22 14:37:46 2019 +1200
Bug 1827811: Redirects
This change cleans up the last of the redirects to artefact/
Update the links for the journal and journal entries for elasticsearch to go to blog
Update the links for the journal and journal entries in consideration of new reqs
Update artefact/artefact with session msg + access denied error
Change-Id: Ied34542632bd4e
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : | #40 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit fb47b8310335515
Author: Lisa Seeto <email address hidden>
Date: Fri Sep 6 09:34:52 2019 +1200
Bug 1827811: UI Task
The change is for making sure the styling is working for the blocks.
* make file(s) to download use bh icons, metadata on new line
* resume attachments to use icons (buggy but apparently necessary)
* show file(s) to download titles on hover over link
* get rid of '-' for file metadata for size etc...
* fix the editing/display descrepancy of tagged/recent posts titles
Change-Id: Iad7bb30944f592
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : | #41 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit 44fcc9c9f72e7a6
Author: Lisa Seeto <email address hidden>
Date: Tue Sep 3 15:51:32 2019 +1200
Bug 1827811: Folder Task
This change adds modal functionality for folders and nested folders
add javascript to the folder_render_self template.
add new class to make space between icon and title/ metadata in modals
fix blockid warning message error
added folder_
creaeted toggleDetailsBtn() to handle js issues
get rid of old style comments link from showing up.
turn off click functions so json sendrequest doesn't call a billion times.
explicitly call dock.show() in the open_modal func.
Change-Id: I938f98c33479a8
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : | #42 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit f8d9ec0a78363bc
Author: Evonne <email address hidden>
Date: Fri Sep 6 10:58:28 2019 +1200
Bug 1827811: UI Task theming
Fixing alignments and hover/active states for assessment buttons.
behatnotneeded
Change-Id: I8774ecc53d5d37
Mahara Bot (dev-mahara) wrote : | #44 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit 1885a18de312350
Author: Lisa Seeto <email address hidden>
Date: Fri Jul 19 11:11:10 2019 +1200
Bug 1827811: Notes block
do not show note contents when displayed in a modal.
if comments not allowed but license metadata is on, display details link and open model with license data.
make small change to setting of licensemetadata smarty variable
get rid of comments/details links and use the new block header instead.
Change-Id: I9183292278d06a
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : | #37 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit f7431a3a3034f58
Author: Lisa Seeto <email address hidden>
Date: Wed Jun 19 13:28:07 2019 +1200
Bug 1827811: Journal changes
Journal changes:
Change the modal contents to not show post contents or attached files.
Recent posts and Tagged posts (if not in full display) now expandable with post contents and comment links.
Attached files on page block now links only to download and not to display further metadata.
Cleaned up artefact.php links.
remove metada that will show on entry on page from also showing in modal
stop background page scrolling when the modal is opened and "Advanced licensing" is selected.
add the blogpost_
get rid of scrolling for data-toggle collapse
add block-header to journal, journal entry
add titles as modal links for recent posts and tagged journal entries
add comments and details icons for recent posts and tagged journal entries
fix things from code review
behatnotneeded
Change-Id: Ic63ebf9fe42e78
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : | #43 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit 8df0061e7619122
Author: Lisa Seeto <email address hidden>
Date: Tue Jul 16 16:53:49 2019 +1200
Bug 1827811: Resume changes
added the modal_link for resume field attachments
fixed the modal)link typo in employmenthistory template
update commentlink to modal_link for goals and skills template
Change-Id: I8b203628dc47d7
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : | #45 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit ece65af480bf518
Author: Lisa Seeto <email address hidden>
Date: Wed Jun 19 17:01:20 2019 +1200
Bug 1827811: Plans block
behatnotneeded
The plan block's 'details' link will only display if license metadata is available.
The link will open the modal with licensing information.
Update how the licensemetadata is set in smarty variable.
remove old details link and use new block header instead.
remove the use of modal functionality, block headers & smarty vars
Change-Id: I5f06d9ba84ff01
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : | #46 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit be7af61faa0f8ca
Author: Lisa Seeto <email address hidden>
Date: Thu May 30 11:07:14 2019 +1200
Bug 1827811: Move "Add comment" and "Details" into modal with "Comments"
create modal using configureblock id, update various templates to use the modal for blocks:
Image
Note
File(s) to download
Journal/blogs
add JS to open/close modal, add comments, delete comments, reply to comments, paginate comments.
add JS to handle url that redirects to view/view.php with modal open and populated.
Further changes to each block will be completed in child patches.
Add segment headings to the modal. Fix display of comments to cover width of modal.
Show comment removed msg in modal/page appropriately.
Fix behat test to close modal before continuing.
do not show existing comments if comments not allowed on artefact.
add validation to modal for comments/
disable image as link when editing for image block
trigger focus on modal for accessibility
close modal on escape keydown event
fix merge conflict in view/view.php
fix merge conflict with master
make configureblock modal play nicely when comment block is on the page
update display of error messages to always show at top of modal so user doesn't have to scroll
add accesibility to close modal on escape key down
handle dock show/hide events for paginated pages in modal during validation
create block-comments-
create details button and custom scss file with js to show and hide header
update comment links to save add comment and details when commentcount == 0
add active-block class to colour the block-header when active
toggle the details-btn active css
add _block-
change class invisble to class d-none
add block-comments-
code review changes
add .no-modal class to links on group home page, dashboard and profile page.
add block header to appear above the title
change assessbtn from anchor tag to button to prevent scrolling.
make all clickable links clickable on page load (ignore assess mode)
fix js issues surrounding assessbtn move and show/hide blocks
change cursor on dashboard, my profile page, group homepage to default when over disabled modal links
update assessbtn to details btn
create tooltip for hover over file download block titles
fix containerElm console log error by not reattaching tinymce unneccessarily.
fix null named attributes issues on e.target when span (icons) instead of anchor is clicked.
Change-Id: I2e5218c040f8a9
Signed-off-by: Lisa Seeto <email address hidden>
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review | #47 |
Patch for "master" branch: https:/
Mahara Bot (dev-mahara) wrote : A change has been merged | #48 |
Reviewed: https:/
Committed: https:/
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master
commit f5ca6b3aa22a682
Author: Evonne <email address hidden>
Date: Wed Sep 18 15:09:27 2019 +1200
Bug 1827811: Journal and resume details buttons overlap
Fixes for journal titles and resume attachments for employment
overlapped by details button.
behatnotneeded
- clean up unneccessary include in export/html/lib.php
- fix html export for the creation of nested folder modals
make sure the nested folders and find all nested modals
- fix elseif typo in block-comments-
- add details bar to image gallery when displayed from folders
- fix notes block empty modal issue due to .tpl typo
- fix image gallery slideshow error where second gallery block
always targets the first
- keep the active block headers showing after pagination update
Change-Id: Icf8d59ad558d74
Rangi Daymond (rangid) wrote : | #49 |
- Education and Employment attachment names not obscured by Details and Comment icons.png Edit (89.6 KiB, image/png)
**Gerrit/Patchset: https:/
**Browser tested: Firefox
**Theme used: Default (Raw)
**DESCRIPTION/
Fixes for journal titles and resume attachments for employment/
**PRECONDITIONS:
1. Logged in user has a portfolio/page that includes:
a) journal blocks (3 types) and journal entries with long titles, and,
b) a resume that includes Education and Employment entries each with attachments with long names
**TEST STEPS:
Step1: Logged in user navigates to their portfolio/page and views each of the following blocks' details and comments modals:
1.1 Journal entries
1.2 Recent journal entries
1.3 Tagged journal entries
2.1 Resume Education entry with one or more attachments
2.2 Resume Employment entry with one or more attachments
Expected result: The entry artefact's name/title is clearly displayed (ie not partially obscured by the Details/Comments icons)
Actual result: As expected (see Resume attachment examples) ✔ Catalyst QA
Changed in mahara: | |
status: | In Progress → Fix Committed |
tags: | added: nominatedfeature |
Changed in mahara: | |
status: | Fix Committed → Fix Released |
Patch for "master" branch: https:/ /reviews. mahara. org/10061