Previously we reloaded sidebar or notebookbar on theme change because
we wanted to load icons to match new colors. This change makes
icons aware of theme change so they automatically switch the URL.
Reload of notebookbar or sidebar is not needed then.
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: Iae0faa904e386765abebbdb8cc6cf3a156605706
in case of list view without levels we don't need
margin in first cell
detect that based on presence of aria-level property
use width: max-content to be sure we don't wrap entries
if not needed
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: Ica98e90dffd1e2b7330eb962cfc8141fa6fe01a1
Use pure CSS to make margins for different levels.
Inspired by: https://www.w3.org/WAI/ARIA/apg/patterns/treegrid/examples/treegrid-1/#ex_label
- reuse arrow from tree view and expander widget
- center verticaly entries in the tree grid / list view
- add pointer cursor for expander arrow
- do not allow to sort tree grid - it makes no sense
Tree grid is a kind of table with headers but rows can be expanded
and have subentries with different "level" in the tree
It can be tested in Writer -> Review -> Manage Changes
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I4d0719a35cf852127378525153eb5b1e5d9961a7
To avoid 2 scrollbars in this dialog better to set a max-height to
this element so it never surpasses the main dialog height
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I939fd5fa3cafb3fcc0721aa2864b262535f622a7
This fixes regression introduce in: Fix Spell checker js Dialog
- fd630d427f
By reading the code, it seems there was a fix for the spelling
suggestion (that comes as an image from core). It fixed the width,
before that the image was being resized to fit the parent's width.
- This fix still works but it ended up affecting any other
.ui-scrollwindow (that is not .formulabar)
- which means that (for **any** text content) the content will not
wrap at all even if it causes overflows and it will occupy
only as much space as it needs and not the full parent's
width.
- Affected dialogs: Sort dialog's ui-scrollwindow was occupying
only enough space to display its contents, leaving a big white
space on the side
The present commit applies the max-content (that should never be
applied to text) solely when we have drawing-area child
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Iaf461663508a70a6a599cbe3e9411026b54531f0
This fixes side effect from commit a1b2091ae6
JSDialog: Dialogs shouldn't be taller than viewport
As mentioned in the above commit, we seemed to have children that are
always bigger than the parent even if there is no elements to be
displayed and to fix that we enforce in multiple places the overflow
hidden: afbf052bb5
- Ideally we wouldn't do this and just let the web browser
figure out if a scroll bar is needed or not. However this
would be a quite risky thing to do without enough testing...so
Best to ensure that other dialogs that have hidden items in the
DOM don't get scrollbars. Examples:
- Writer -> Format -> Character
- Calc -> Data -> Validity
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: Ia2625bd7e45214aa43ae96e95a149e0f78c9cd7c
Not sure if all of them can be used now, but these 5 i tested:
insertion, deletion, cut, paste, format
Signed-off-by: Attila Szűcs <attila.szucs@collabora.com>
Change-Id: I4a6c974817d58455f01f106232c9baa66fe9d58c
If postmessage to insert custom button will arrive
early before we initialized notebookbar it could
fail in insertion into compact mode toolbar due to
not existing toolbar.
This prevents us from that error. It will be not added
at the time of postmessage execution
but we remember all the custom buttons in special
array, so when UI will be initialized it will be added there.
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I07d86db49a50be70b0c091a66cf5e4be049211ce
Without that integrator doesn't know what happened.
We were silently ignoring messages.
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I897a95b343a1b436745816ccbef7656f30981112
With this fix, user can open a new commen popup without closing a previous new comment popup.
Without this, a new comment section lingers at the background and new comment command fails until user refreshes page.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: If7f151eb11d3f5d26dae2ca6331eb5dc0dfd9881
- Condition modified for getPrecision.
- it was not handling the cases for dot value less then 0.
Signed-off-by: Darshan-upadhyay1110 <darshan.upadhyay@collabora.com>
Change-Id: I740f86e9477656d371ba3b8793bff99671444d7c
- jsdialog requires to create in the handler single root node
with id of the widget
- remove old style update by _refreshSidebar
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I8ce6e832b2d27d25f54e9bf636572b8c97d3a7c1
queryselectorall('something').queryselectorall('something else') fails.
^ This JS error is fixed.
Also we don't have specific images for German help page. So we removed the "de" from that language list.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: I69b56e8f4bd2b92835c4aca9c2284750f3df9455
for font selector and font size...
this makes us sure we initialize everything in Impress and Draw
too in case of reconnect
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I67ee15f470fa43965f6fbd91ae45cd330ea449aa
problem:
earlier restriction/lock status was sent to kit via browser,
now client session send this status directily to the kit.
this design will require less communication between server and browser
Signed-off-by: Pranam Lashkari <lpranam@collabora.com>
Change-Id: I6b830f30fb326a5e6637e345250893cbba101de6
TypeError was noticed where container was undefined.
This was case where snackbar was closed.
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: Id465723e01bc2cec2e20056103a918e53abf6d09
problem:
this caused problem in autosaved comments,
when comment list is empty autosaved comments may be closed when focus is lost
partial fix: #7421
Signed-off-by: Pranam Lashkari <lpranam@collabora.com>
Change-Id: I1fc2d087828ce44495001f97b5323dffb8adb2ef
Also add a check for failed attempts of new comments.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: I50a9981cf51475d6008e936e6e28573156a8383a
Set up accessible text through aria-description for describing how to
enable accessibility support for text content when it is disabled.
The description is reported by the screen reader on document load and
on explicit request to read current text content.
Signed-off-by: Marco Cecchetti <marco.cecchetti@collabora.com>
Change-Id: I439b7703c15d3b38c39181d27c11da438834c414
we emit lots of warnings which are not important
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I59027c4d848a98ba3d9e8f4a3feafd3fcb62dec2
