- Adjust left position and
- Makes sure option is not getting weird values from web browser
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Ia753baf8066914543388c80827b2fe4c8ed26c3b
Fix inconsistency between ui-treeview-expander and ui-expander
- Expanded: pointing down
- Collapsed: pointing to the right
- Also make sure the width doesn't change between states
(use 'V' in both)
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Ib4b72d6f8acca33fcb67981ff9b6fd8095012ead
- Fix Borders
- Make sure the borders that divide the document container and its
surroundings are always visible at all times not matter what is open
(sidebar) or what is visible (status bar, ruler)
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Idea765f4438981af66bb760f25f7c553be7da728
- Dropdown (select element aka listbox) Do not use min or max widths
- Use CSS vars for text colours
- Add hover status to buttons
- Force to also add id references since these lack the unotoolbutton
class
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I480deb651a83b44a8a5e4170e105b4dc2e900ec5
Plus make it start from the top and ending at the bottom
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: If8c89a248c467e50186a82bc191200a60e2823e1
Via CSS var to make sure we use them consistently throughout
jscomponents. In some browsers it was being set with serif specimen
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I418ad3a313c394e526b6e4e1c46ebe3616212d40
Toolbar-down was being covered by #map
- Set toolbar-down with the same z-index and ruler since both should be
above map
- Update readme
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: Ic358fb5f7bd722806cf22fdad0289f8a54fa6e1e
- Sidebar parents: be sure it's container childern is always full width
- Sidebar parent: Fix margins vs paddings
- Expander: Decrease color contrast so to avoid to be in the way of the
actual components
- Fix expanders (arrow) position and paddings
- Use CSS vars
- Fix and style listbox and spinfield to be slimmer and more compact
than the regular ones (crucial here to effectivly use space)
- Some of these things should be ported top the general jsdialog css
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I50def4b41144d56a3cc2de41b3ed00ed550fa74d
- Add lateral animation so the users understands that it is in fact a
sidebar
- Also that animation should help with the way toolbars and sidebar
appear, each on on their own time causing the sidebar to appear first
and abruptly (appearing to be floating for a couple of milliseconds)
It would be good to have a longer animation time when loading the page
then when simply pressing the button so something like :
- Currently the default 1000 is somehow skipped
- this.map.sidebar = L.control.sidebar({animSpeed: 200});
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I95b6bf51900449e85c24cee6d5d47f8af9a0f134
So it can be easy to analyze the backtrace of the
failing assertion.
Change-Id: Icdcf55b1404f858eba1d499bd65719a0e2da21db
Signed-off-by: Henry Castro <hcastro@collabora.com>
when calling hasTableSelection(). Do not depend on stray/duplicate
tableselection messages that can ensure a valid _currentTableData
member of TileLayer.
Signed-off-by: Dennis Francis <dennis.francis@collabora.com>
Change-Id: I957fdfc43c1435ca98eb204dcb3ec1e76b85ce89
Otherwise the unit test will take too much time
to fail.
Change-Id: I9a85a79ad2ced5d220394cdf71a7f703265e50ef
Signed-off-by: Henry Castro <hcastro@collabora.com>
When adding a freshly arrived textinput message to the queue, check if
there is an outstanding one queued, and if so, remove it, and prefix
its text to the text in the fresh message. Don't do this if there are
messages inbetween that affect what the textinput messages do
semantically, including textinput messages from other clients.
Signed-off-by: Tor Lillqvist <tml@collabora.com>
Change-Id: I712eebfb10d410bb424157c8df46b2848a236d88
For some servers we receive failure with HTTP 403 Forbidden in WOPI::CheckFileInfo
"Reason: The client software did not provide a hostname using Server
Name Indication (SNI), which is required to access this server"
fixes#2771 : https://github.com/CollaboraOnline/online/issues/2771
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I761b179580481f8882a4526c1d8be4f1c14ad929
Event orders or propagation of events are different while using cypress tests.
Newly created comment losts focus while menu is being closed. This scenario is very difficult to reproduce by user (i couldn't).
I added a case for cypress test.
Since this behaviour would cause a regression in production, it is activated only for cypress tests.
Regression without if statement would be: Click insert comment button via menu, open and click the same button once again. Since the first comment is not deleted, the second comment will cause an error (can't add sections with the same name).
Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: Iaecdd36b56664e207b3a74de5d2b013a845f3fba
if make is killed or interrupted during the execution
of their recipes, the target is not deleted. So there is
an opportunity to check the logs, why are taking to long
time to finish the test.
Change-Id: I6e8176a48ff721ecb2b2efc01b1f149907dd359b
Signed-off-by: Henry Castro <hcastro@collabora.com>
Add profile zone around kitPoll function and SocketPoll constructor to
improve trace event generation
Signed-off-by: Gopi Krishna Menon <krishnagopi487.github@outlook.com>
Change-Id: Ia46e9add59a57935997649fe39861a8d851e1ff0
"Async" here refers a specific kind of data in the Trace Event JSON
format, not asynchrony in general.
Actually, we don't have code that would generate such yet in
loleaflet. Just the createAsyncTraceEvent function in Socket.js. Quite
possibly the "Complete" events (a.k.a. ProfileZones) are all we need.
Signed-off-by: Tor Lillqvist <tml@collabora.com>
Change-Id: I611ccd4b964bcacdd1050893f70dbdb813ee3c34
At least I didn't fully understand what the old code actually was
checking.
Also, use sizeof instead of magic numbers.
Signed-off-by: Tor Lillqvist <tml@collabora.com>
Change-Id: I01602a2e256b1fbe39793c9f1439e025461c5a72
Our JS code does not send 'textinput' messages with a 'type' parameter
either, so it is pointless to do it in a unit test.
(We still have code that checks for the 'type' parameter in received
'textinput' messages. That is pointless, too, and should be removed.)
Also, document what testWriterTyping() apparently is doing.
Signed-off-by: Tor Lillqvist <tml@collabora.com>
Change-Id: I4954379339f14e4a310a76788521001048d84e37
Since a99edb5d29 this gets a different lifecycle, and is often
joined and gone by the time we get to cleanup.
Change-Id: Ic21ca837d484a9a853c7f8f683e876406b00a943
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>
Set min-height property to prevent exceeding the available size.
slide-sorter: Set max-height to prevent over-sizing relative its parent.
Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: Ib8c50bc9f6f6ac4311cefa3b0e016da728efe95b
* Use lineHeight rule for centering the elements on mobile.
* Use parent element's rules for setting the styles of img and preview elements.
* Disable mobile-slide-sorter. It is not in use for a long while.
Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: I06517467f269669c70294fb5a4dd0960eb16feb5
It seems that fitWidthZoom function is impress-only.
I will turn back to this later.
Signed-off-by: Gökay ŞATIR <gokaysatir@gmail.com>
Change-Id: I95b02d53299bf1b8ce7daf31121b67dccd1c817c
